#62546 closed defect (bug) (invalid)
Duplicate menu-top-first class in admin menu
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Severity: | trivial | Version: | |
| Component: | Administration | Keywords: | has-patch has-unit-tests close |
| 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 (3)
Change History (17)
#3
@
13 months 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
@
13 months 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.
13 months ago
#5
- Keywords has-patch has-unit-tests added
Trac ticket: https://core.trac.wordpress.org/ticket/62546
#6
@
13 months 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
#7
@
12 months ago
Hello @kkmuffme,
I tested the issue and found that the class is added twice when the WooCommerce plugin is activated. When I deactivate or remove WooCommerce, the issue is resolved. Therefore, this is not a WordPress core issue.
Environment with woocommerce
- WordPress: 6.7.1
- PHP: 8.1.2-1ubuntu2.20
- Server: Apache/2.4.52 (Ubuntu)
- Database: mysqli (Server: 8.0.40-0ubuntu0.22.04.1 / Client: mysqlnd 8.1.2-1ubuntu2.20)
- Browser: Chrome 131.0.0.0
- OS: Linux
- Theme: Twenty Twenty-Five 1.0
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
- WooCommerce 9.5.1
Result: ✅ Issue occurs (reproduced).
Environment without woocommerce
- WordPress: 6.7.1
- PHP: 8.1.2-1ubuntu2.20
- Server: Apache/2.4.52 (Ubuntu)
- Database: mysqli (Server: 8.0.40-0ubuntu0.22.04.1 / Client: mysqlnd 8.1.2-1ubuntu2.20)
- Browser: Chrome 131.0.0.0
- OS: Linux
- Theme: Twenty Twenty-Five 1.0
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
Result: ❌ Issue does not occur.
#8
@
12 months ago
- Milestone changed from Awaiting Review to 6.8
- Owner set to SergeyBiryukov
- Status changed from new to reviewing
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
9 months ago
#10
@
9 months ago
- Keywords close added
As per today's bug scrub, it appears this ticket is not directly related to Core. Let's wait for a confirmation though. I'm adding the close keyword in the meantime.
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
9 months ago
#12
@
9 months ago
- Milestone 6.8 deleted
- Resolution set to invalid
- Status changed from reviewing to closed
I'm closing this ticket as it looks like it's not related to WP Core.Please open a ticket on WooCommerce side.
#13
@
9 months ago
https://github.com/woocommerce/woocommerce/issues/56466 was opened porting this over to WooCommerce.
#14
@
9 months ago
Noting that Woo closed the issue there, so if you have insights for them please comment on https://github.com/woocommerce/woocommerce/issues/56466
My mistake. You meant the side admin menu.