Make WordPress Core

Opened 7 weeks ago

Closed 3 weeks ago

#64692 closed defect (bug) (fixed)

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

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

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 weeks ago.
Current situation on fr_FR locale
Capture d’écran 2026-02-21 à 07.47.19.png (339.1 KB) - added by audrasjb 7 weeks ago.
After removing floats
Capture d’écran 2026-02-21 à 09.18.27.png (268.2 KB) - added by audrasjb 7 weeks ago.
Updated PR with width 100%

Download all attachments as: .zip

Change History (16)

@audrasjb
7 weeks ago

Current situation on fr_FR locale

@audrasjb
7 weeks ago

After removing floats

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


7 weeks ago
#1

  • Keywords has-patch added

#2 follow-up: @huzaifaalmesbah
7 weeks 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 weeks 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 weeks ago

Updated PR with width 100%

@shailu25 commented on PR #10997:


7 weeks 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:


5 weeks 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
4 weeks 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.


4 weeks 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.


4 weeks ago

#9 @ozgursar
4 weeks 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 4 weeks ago by ozgursar (previous) (diff)

@huzaifaalmesbah commented on PR #11273:


4 weeks 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:


4 weeks 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
4 weeks ago

  • Version set to trunk

#13 @audrasjb
3 weeks ago

  • Owner set to audrasjb
  • Resolution set to fixed
  • Status changed from new to closed

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.