Make WordPress Core

Opened 7 months ago

Closed 6 months ago

#64692 closed defect (bug) (fixed)

[admin-reskin] Remove floats from custom links section in the Nav Menus screen

Reported by: audrasjb Owned by: audrasjb
Priority: normal Milestone: 7.0
Component: Menus Version: 7.0
Severity: normal Keywords: has-screenshots has-patch
Cc: Focuses: ui, css

Description

With the admin reskin, there are some issues on the custom links section of the nav menu screen in some Locales. See screenshots below for the fr_FR locale.

I suggest to simply remove the existing float css properties to avoid such issues.

Attachments (3)

Capture d’écran 2026-02-21 à 07.41.56.png (326.9 KB ) - added by audrasjb 7 months ago.
Current situation on fr_FR locale
Capture d’écran 2026-02-21 à 07.47.19.png (339.1 KB ) - added by audrasjb 7 months ago.
After removing floats
Capture d’écran 2026-02-21 à 09.18.27.png (268.2 KB ) - added by audrasjb 7 months ago.
Updated PR with width 100%

Download all attachments as: .zip

Change History (16)

@audrasjb
7 months ago

Current situation on fr_FR locale

@audrasjb
7 months ago

After removing floats

This ticket was mentioned in PR #10997 on WordPress/wordpress-develop by @audrasjb.


7 months ago
#1

  • Keywords has-patch added

#2 follow-up: @huzaifaalmesbah
7 months ago

I tested it with Belarusian (Беларуская мова) and saw the same issue. After applying the PR #10997 , the issue was fixed.

However, Maybe it would look better if we use:

.menu-item-textbox {
    width: 100%;
}

Since the label and the textbox are currently on separate rows.

Before Apply Patch After Apply Patch ✅
https://i.ibb.co/Txv3kbPz/Huzaifa-20260221133409.png https://i.ibb.co/0Vm1XcSd/Huzaifa-20260221133646.png

#3 in reply to: ↑ 2 @audrasjb
7 months ago

Replying to huzaifaalmesbah:

However, Maybe it would look better if we use:

.menu-item-textbox {
    width: 100%;
}

Since the label and the textbox are currently on separate rows.

Right, let's do that. PR updated.

@audrasjb
7 months ago

Updated PR with width 100%

@shailu25 commented on PR #10997:


7 months ago
#4

I believe that if we set the input box width to 100%, we may also need to adjust the button alignment, as it does not appear to be properly aligned at the moment. It should support alignment options such as right, center, or left, as appropriate.

Just sharing my thoughts.

Currently, it appears as shown here: https://prnt.sc/GRIgTITSt3Zr

@audrasjb commented on PR #10997:


6 months ago
#5

Thanks for the comments! If anyone is able to rebuild a PR for this, it would be really appreciated because I'm not sure I have time to update this one in time before RC1

#6 @khushi1501
6 months ago

Tested this on WordPress 7.0 Beta 5 and the patch works as expected.

The layout in the Custom Links section looks better after removing floats. I’m also in favor of adjusting the button alignment, as it improves the overall consistency of the UI.

Before applying the patch:-
https://prnt.sc/gZZ_JZ8RlCfI

After applying the patch:-
https://prnt.sc/F-0He1AnD-hk

This ticket was mentioned in PR #11273 on WordPress/wordpress-develop by @nikunj8866.


6 months ago
#7

Trac ticket: https://core.trac.wordpress.org/ticket/64692

## Use of AI Tools

This ticket was mentioned in Slack in #core-test by nikunj8866. View the logs.


6 months ago

#9 @ozgursar
6 months ago

Patch Testing Report

Patch Tested: https://github.com/WordPress/wordpress-develop/pull/11273
with language Afrikaans

Environment

  • WordPress: 7.0-beta5-61991-src
  • PHP: 8.2.29
  • Server: nginx/1.29.4
  • Database: mysqli (Server: 8.4.7 / Client: mysqlnd 8.2.29)
  • Browser: Opera
  • OS: macOS
  • Theme: Twenty Fifteen 4.1
  • MU Plugins: None activated
  • Plugins:
    • Code Snippets 3.9.5
    • Test Reports 1.2.1

Steps taken

  1. Choose a locale
  2. View Menus > Appearance and extend Custom Links
  3. Observe layout bug due to floats
  4. Apply patch
  5. Observe labels properly located above the input fields and their width set to 100%
  6. ✅ Patch is solving the problem

Screenshots/Screencast with results

Before
https://i.imgur.com/Xt2lTZI.png

After
https://i.imgur.com/3jPvi5o.png

Last edited 6 months ago by ozgursar (previous) (diff)

@huzaifaalmesbah commented on PR #11273:


6 months ago
#10

Looks good to me ✅

English Français
https://github.com/user-attachments/assets/eeeaac6e-3d67-49b3-951d-ed4111da81aa https://github.com/user-attachments/assets/d93571a2-b40f-491f-9b6c-19ab07079361

@audrasjb commented on PR #11273:


6 months ago
#11

This looks all good to me. Also tested on small screen and with fr_FR locale.

https://github.com/user-attachments/assets/ff1ecd57-0a21-458c-b425-1a6e1fc351c6
https://github.com/user-attachments/assets/6d48bddf-b40d-4f4b-ac51-2f08f917bbec
https://github.com/user-attachments/assets/a5c958c5-42bc-49fb-a807-ae78cd0eed47
https://github.com/user-attachments/assets/6e036a49-21a3-44c7-b353-69793f51a46f
https://github.com/user-attachments/assets/8aa45f0a-9698-4728-b08f-b4e4d8b2b292

#12 @audrasjb
6 months ago

  • Versiontrunk

#13 @audrasjb
6 months ago

  • Owner set to audrasjb
  • Resolutionfixed
  • Status newclosed

In 62097:

Menus: Remove floats from custom links section in the Nav Menus screen.

With the new admin design, there were some issues with floating elements in the custom links section of the classic Menu screen. This changeset simplifies the CSS implementation by removing floats and wp-clearfix classes from some elements in favor of a more robust implementation based on flex positionning.

Props audrasjb, huzaifaalmesbah, shailu25, nikunj8866, joedolson, khushi1501, ozgursar.
Fixes #64692.
See #64308.

Note: See TracTickets for help on using tickets.