WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#26083 closed defect (bug) (fixed)

sidebar submenu overlapped by current theme block after theme activation

Reported by: senlin's profile senlin Owned by:
Milestone: 3.8 Priority: normal
Severity: normal Version: 3.8
Component: Customize Keywords: has-patch commit
Focuses: ui Cc:

Description

Steps to reproduce:

  1. if on Firefox/Chrome, make sure not to use Firebug/Inspect Element
  2. activate a new theme
  3. hover over "Posts", "Media" or "Pages" to see that their sub-menus become hidden

screenshot:
https://www.evernote.com/shard/s19/sh/121b9540-2f6d-4602-9e65-da126a70298b/9c7a7a0a5337680271b0d0fef53bb9f1/res/dd73387d-b9f8-471f-ab1f-45e3e470581f/skitch.jpg

I have tried to adjust the obvious one (add z-index: 11;to line 1788 on wp-admin.css #adminmenu .wp-not-current-submenu .wp-submenu), but that does not have the desired outcome.

Attachments (2)

26083.patch​ (433 bytes) - added by senlin 12 years ago.
adjust z-index
26083.2.patch​ (408 bytes) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (7)

#1 @SergeyBiryukov
12 years ago

  • Keywords ui-focus added
  • Milestone changed from Awaiting Review to 3.8

Confirmed.

@senlin
12 years ago

adjust z-index

#2 @senlin
12 years ago

  • Keywords has-patch added

@SergeyBiryukov
12 years ago

#3 follow-up: @SergeyBiryukov
12 years ago

  • Keywords commit added

In 26083.patch​, the first selector becomes redundant.

I guess we can just merge that rule with the previous block (.theme.active .theme-actions): 26083.2.patch​. Didn't notice any side effects.

#4 in reply to: ↑ 3 @senlin
12 years ago

Replying to SergeyBiryukov:

In 26083.patch​, the first selector becomes redundant.

I'm sorry, I didn't realise that the first selector was redundant.

#5 @SergeyBiryukov
12 years ago

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [26280].

Note: See TracTickets for help on using tickets.