Opacity not applied to admin menu icon when it selected

[chandrapatel]

Opacity not applied to single menu with custom menu logo when it selected.

But it work fine where menu has sub-menu and following css applied for it.

#adminmenu li.menu-top:hover .wp-menu-image img, #adminmenu li.wp-has-current-submenu .wp-menu-image img {
  opacity: 1;
}

So here css rule missing for single menu with custom menu logo.

When I applied following css then it work fine.

#adminmenu li.menu-top.current .wp-menu-image img {
  opacity: 1;
}

[chandrapatel]

Added patch

[chandrapatel]

screenshot to show issue.

[sabernhardt]

37712.patch​ still applies and makes the image fully opaque when its menu item is the current page.

Because dashicons and background images show at full opacity even when not current/mouseover now, it may be better to make the image fully opaque all the time as well.

[sabernhardt]

option to keep the image at full opacity

[rehanali]

Patch Added

[SirLouen]

Reproduction Report

Description

❌This report can't validate that the issue can be reproduced.

Environment

• WordPress: 6.9-alpha-60093-src
• PHP: 8.2.28
• Server: nginx/1.29.0
• Database: mysqli (Server: 8.4.5 / Client: mysqlnd 8.2.28)
• Browser: Chrome 137.0.0.0
• OS: Windows 10/11
• Theme: Twenty Twenty-Five 1.2
• MU Plugins: None activated
• Plugins:
  • Test Reports 1.2.0

Actual Results

1. ❌ Error condition is not occurring

Additional Notes

• I can't reproduce this. Was this being tested back in the day in Internet Explorer? If anyone can throw a screencast reproducing the issue (instead of some screenshots), it would be good to proceed with this.
• Although I prefer the patch by @sabernhardt, better removing than adding more classes to the mix. I will not do any harm, regardless that it can be reproduced or not.
• For now adding reporter-feedback and close, worksforme preemptively.

Supplemental Artifacts

Opacity on current selected menu showing at 100%.

[sabernhardt]

Transcoder has its own CSS for the current menu item in rt-transcoder-admin.css now:

#adminmenu li.menu-top.current .wp-menu-image img {
  opacity: 1;
}

Two TeamUpdraft plugins use PNG images that still show the lower opacity:

• UpdraftPlus only has the top-level link, which has the partially transparent image whether that is the current page or not. The image has full opacity when hovering over the link.
• All-In-One Security (AIOS) has a submenu, so the image shows at full opacity when one of its pages is current (without needing to hover over it).

[sabernhardt]

opacity of PNG images in the admin menu (current)

[SirLouen]

Replying to sabernhardt:

Two TeamUpdraft plugins use PNG images that still show the lower opacity:

• UpdraftPlus only has the top-level link, which has the partially transparent image whether that is the current page or not. The image has full opacity when hovering over the link.
• All-In-One Security (AIOS) has a submenu, so the image shows at full opacity when one of its pages is current (without needing to hover over it).

This looks pretty good to me

And its taking default CSS without patch

Judging by this image you posted:

I see that the only problem is in no-hover for UpDraft Plus, but this is not happening with WP Security that is the one I tried here.

So I can assume that this is the bug? Or is the no-hover in WP Security without Opacity?

Not sure now which should be the expected behaviour here.