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.

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.