Opened 2 weeks ago
Last modified 2 weeks ago
#62546 new defect (bug)
Duplicate menu-top-first class in admin menu
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Awaiting Review | Priority: | normal |
| Severity: | trivial | Version: | |
| Component: | Administration | Keywords: | has-patch has-unit-tests |
| Focuses: | Cc: |
Description
the "menu-top-first" class is there twice (just open /wp-admin/)
<li class="wp-first-item wp-has-submenu wp-has-current-submenu wp-menu-open menu-top menu-top-first menu-icon-dashboard menu-top-first" id="menu-dashboard">
Same issue also here:
<a href='index.php' class="wp-first-item wp-has-submenu wp-has-current-submenu wp-menu-open menu-top menu-top-first menu-icon-dashboard menu-top-first">
Attachments (1)
Change History (7)
#3
@
2 weeks ago
- Keywords reporter-feedback added
Hi @kkmuffme.,
I cannot find the exact issue that you have mentioned. The class you have mentioned is not duplicated. Please check the attached screenshot
#4
@
2 weeks ago
Hey @kkmuffme,
As @karthickmurugan demonstrated, even I couldn't find the duplication of the "menu-top-first" class, neither in the codebase nor after inspecting the admin menu page.
Can you please provide a specific location or screenshot of the page where you have encountered this issue?
This ticket was mentioned in PR #7888 on WordPress/wordpress-develop by @kkmuffme.
2 weeks ago
#5
- Keywords has-patch has-unit-tests added
Trac ticket: https://core.trac.wordpress.org/ticket/62546
#6
@
2 weeks ago
- Keywords reporter-feedback removed
@karthickmurugan it depends on the which menu's are displayed.
Issue arises from https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-admin/menu.php#L24 which gets the duplicate class added from https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-admin/includes/menu.php#L234 or
https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-admin/includes/menu.php#L251
Created a PR to fix it
My mistake. You meant the side admin menu.