Site icon: Don't force cropping

[jorbin]

Do not force users to go through the cropping flow if the image is the correct ratio. This was fixed in the customizer in r59197, but that change needs to be ported to the site-options form of the site icon.

Trac ticket: #64020

[gulamdastgir04]

I have tested the patch in the playground using a 512x512 pixel square image as the site icon. When added through the Site Icon option, it successfully bypasses the image cropping flow.

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: ​https://github.com/WordPress/wordpress-develop/pull/10179.diff

Environment

• WordPress: 6.9-alpha-20251007.105636
• PHP: 8.3.25-dev
• Server: PHP.wasm
• Database: WP_SQLite_Driver (Server: 5.5 / Client: 3.40.1)
• Browser: Chrome 141.0.0.0
• OS: Linux
• Theme: Twenty Twenty-Five 1.3
• MU Plugins: None activated
• Plugins:
  • Test Reports 1.2.0

Actual Results

✅ Issue resolved with patch.

[kirasong]

screencast - customizer as currently in trunk

[kirasong]

screencast - settings as currently in trunk

[kirasong]

screencast - settings after applying 10179.diff

[kirasong]

I tested the patch, and it does skip the cropping step, but the behavior is different than in the customizer, since it doesn't present the "skip cropping" button -- but instead assumes the user would like to skip.

As a user, I prefer this behavior, but it is a departure from the behavior in the customizer.

I've attached a before and after for customizer and settings.

Customizer Before: customizer - before .mov​
Settings Before: settings - before.mov​
Settings After applying ​10189.diff: settings - after.mov​

[gulamdastgir04]

@kirasong I agree - skipping the cropping step automatically feels like a smoother and more intuitive experience, even though it differs slightly from the Customizer flow. Having to crop an image that’s already the perfect size can be a real headache for users, so this behavior is definitely a welcome improvement.

[gulamdastgir04]

This improvement definitely deserves to be included in the 6.9 release. It feels user-friendly and aligns well with the current UX direction.

[jorbin]

The behavior should be consistent with the customizer. Just because something is square when uploaded doesn't mean that the site admin intends to use that exact square as the site icon.

[wildworks]

It seems we haven't yet reached an agreement on whether to proceed with ​PR 10179. Since the RC1 release is next week, I'd like to punt it to 7.0.

If an agreement is reached on the approach, please revert the milestone back to 6.9, and feel free to move forward.