Add Themes page in UI issue

[sumitsingh]

Hi ,

i have found some small issue on IPhone on add new theme page on admin side.

More information you can see mentioned screenshot.

Thank you

[sumitsingh]

Your WordPress.org username: on button alignment issue

[sumitsingh]

Edit Filters screen

[gauravtiwari]

iPhone uses ubuntu font?

[sumitsingh]

Hi,

What the reply 😂🤣

Any device in check responsive on any Operating system by Developer. I think you are not a developer otherwise you can understand with screenshot.

Replying to gauravtiwari:

iPhone uses ubuntu font?

[gauravtiwari]

No need to get offended. The thing is that virtual devices and dev tools' device screenshots work differently than original devices.

My iPhone uses SFUi font and I cannot see this problem yet.

Can you please confirm which iOS version you are using? Maybe your safari is outdated.

Replying to sumitsingh:

Hi,

What the reply 😂🤣

Any device in check responsive on any Operating system by Developer. I think you are not a developer otherwise you can understand with screenshot.

Replying to gauravtiwari:

iPhone uses ubuntu font?

[sabernhardt]

The filter tags list could be fixed by adding display: inline-block, and I would prefer removing the 11px font size.

I also think the username label, input and button would be better stacked vertically—with a margin between them—at the 782px breakpoint.

[sumitsingh]

Any update abouth this ticket Guys?

Trac ticket: (https://core.trac.wordpress.org/ticket/53314)](https://core.trac.wordpress.org/ticket/53314)

#### After Fix:

[Benjamin_Zekavica]

@sumitsingh I fixed it by using flexbox. See here: ​https://github.com/WordPress/wordpress-develop/pull/6874

[krupajnanda]

Hi,

Test Report

Description

This report validates whether the indicated PR works as expected.

PR tested: ​https://github.com/WordPress/wordpress-develop/pull/6874

Environment

• WordPress: 6.6-beta3-58440-src
• PHP: 8.2.15
• Server: nginx/1.25.3
• Database: mysqli (Server: 8.0.36 / Client: mysqlnd 8.2.15)
• Browser: Chrome 126.0.0.0
• OS: macOS
• Theme: Twenty Twenty-Four 1.1
• MU Plugins:
  • Safe Autoloaded Options Limit Test (MU Plugin) 1.0
• Plugins:
  • Test Reports 1.1.0

Actual Results

1. ✅ Issue resolved with added fix in the given PR.

I tested this with multiple devices and looks like Pr works as intended.

Supplemental Artifacts

Add as Attachment

[Benjamin_Zekavica]

@krupajnanda Thank you so much for testing :) You're awesome, good job.
@audrasjb I think it's ready to merge to the Core SVN, or you have some feedback?

@sabernhardt Thank you :) I will update it later and submit a new patch

[Benjamin_Zekavica]

@sabernhardt I approved your code suggestions, I hope now it's fine ?

[krupajnanda]

@krupajnanda Thank you so much for testing :) You're awesome, good job.

Thank you! 🙇‍♀️

@Benjamin_Zekavica Please let me if this is required an another round for testing.

[Benjamin_Zekavica]

@krupajnanda It would be good to test it again with the new version of the patch. Thank you :)

@sabernhardt I approved your changes :) I hope now it looks better?

[Benjamin_Zekavica]

@audrasjb Please, do you can review it?

@sabernhardt Thank you so much for your review :)

[Benjamin_Zekavica]

Code Changes are approved and it was final tested:
​https://github.com/WordPress/wordpress-develop/pull/6874#pullrequestreview-2305199599

For detailed Screenshots, please see the PR.
It's ready for merging.

Thank you @sabernhardt and @krupajnanda for Testing and Code Review and @sumitsingh for reporting this bug.

@audrasjb @williampatton @SergeyBiryukov Can anyone merge it please?

[SergeyBiryukov]

In 59022:

Themes: Improve the alignment of feature filters and inputs on Add Themes screen.

Follow-up to [35527], [38640], [40797].

Props Benjamin_Zekavica, sabernhardt, sumitsingh, gauravtiwari, krupajnanda, audrasjb, SergeyBiryukov.
Fixes #53314.

Thanks for the PR! Merged in r59022.