WordPress.org
Get WordPress

Make WordPress Core

Opened 8 years ago

Last modified 4 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 4 years ago.
option to keep the image at full opacity
#37712.patch​ (457 bytes) - added by rehanali 3 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
8 years ago

  • Keywords has-patch added

#3 @swissspidy
8 years ago

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

#4 @sabernhardt
4 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
4 years ago

option to keep the image at full opacity

@rehanali
3 years ago

Patch Added

Note: See TracTickets for help on using tickets.