Make WordPress Core

Opened 6 years ago

Last modified 4 years ago

#47165 new enhancement

Admin menu: submenu for active menu doesn't have box-shadow

Reported by: man4toman's profile man4toman Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-screenshots has-patch needs-testing dev-feedback
Focuses: ui Cc:

Description

Submenu for active menu doesn't have box-shadow, please check screenshots.

Attachments (3)

01.png (9.6 KB) - added by man4toman 6 years ago.
02.png (10.3 KB) - added by man4toman 6 years ago.
47165.diff (300 bytes) - added by mukesh27 6 years ago.
Patch.

Download all attachments as: .zip

Change History (11)

@man4toman
6 years ago

@man4toman
6 years ago

#1 @man4toman
6 years ago

Maybe relate with: #47132

Version 0, edited 6 years ago by man4toman (next)

@mukesh27
6 years ago

Patch.

#2 @mukesh27
6 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

This ticket was mentioned in Slack in #design by estelaris. View the logs.


5 years ago

#4 @kjellr
5 years ago

Thanks for the patch, @mukesh27! I just tested, and I think the fix will need to be slightly more complicated: I'm seeing the drop shadow applied when the menu is expanded as well now:

https://cldup.com/LuVi7dCWsn.png

Ideally, that shadow should only show up when the left menu is collapsed, and when the active item is opened up.

#5 @nrqsnchz
5 years ago

Hey @mukesh27! I can confirm what Kjell commented. While the patch adds the missing drop shadow to the flyout menu, it's also adding it to the expanded menu.

https://cldup.com/NW5ileJTcX.png

#6 @JavierCasares
5 years ago

Hello, I tried this and, I think it works...

#adminmenu .wp-has-current-submenu.opensub .wp-submenu
{
  box-shadow: 0 3px 5px rgba(0,0,0,.2);
}

This ticket was mentioned in Slack in #design by estelaris. View the logs.


4 years ago

#8 @anyssa
4 years ago

  • Keywords dev-feedback added; needs-design-feedback removed

As discussed today during design triage, I'm removing the keyword needs-design-feedback, since it seems to be a CSS selector issue at this point.

Note: See TracTickets for help on using tickets.