Make WordPress Core

Opened 8 years ago

Last modified 3 years ago

#37712 new defect (bug)

Opacity not applied to admin menu icon when it selected

Reported by: chandrapatel's profile chandrapatel Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch needs-testing
Focuses: ui, css Cc:

Description

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;
}

Attachments (4)

37712.patch (503 bytes) - added by chandrapatel 8 years ago.
Added patch
opacity-issue-in-menu.png (9.3 KB) - added by chandrapatel 8 years ago.
screenshot to show issue.
37712.1.patch (654 bytes) - added by sabernhardt 3 years ago.
option to keep the image at full opacity
#37712.patch (457 bytes) - added by rehanali 2 years ago.
Patch Added

Download all attachments as: .zip

Change History (8)

@chandrapatel
8 years ago

Added patch

@chandrapatel
8 years ago

screenshot to show issue.

#1 @swissspidy
8 years ago

  • Component changed from Menus to Administration
  • Focuses administration removed
  • Summary changed from Opacity not applied to single menu with custom menu logo when it selected to Opacity not applied to admin menu icon when it selected
  • Version 4.6 deleted

#2 @Presskopp
7 years ago

  • Keywords has-patch added

#3 @swissspidy
7 years ago

  • Keywords needs-testing added
  • Milestone changed from Awaiting Review to Future Release

#4 @sabernhardt
3 years ago

  • Focuses css added

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
3 years ago

option to keep the image at full opacity

@rehanali
2 years ago

Patch Added

Note: See TracTickets for help on using tickets.