WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 22 months ago

#20300 closed defect (bug) (fixed)

Small bug ... change Z-index of Widgets

Reported by: alexvorn2 Owned by: markjaquith
Milestone: 3.5 Priority: normal
Severity: minor Version:
Component: UI Keywords: has-patch commit 3.5-early
Focuses: Cc:

Description

http://wpimpact.com/wp-content/uploads/2012/03/Untitled-22.png
Replace z-index form 5 to 100 for Widgets

Attachments (2)

20300-1.diff (440 bytes) - added by jkudish 2 years ago.
changes z-index to 100 for draggable widgets
20300-2.diff (441 bytes) - added by jkudish 2 years ago.
changes z-index to 1001 instead of 100

Download all attachments as: .zip

Change History (9)

comment:1 helenyhou2 years ago

  • Component changed from Template to UI

jkudish2 years ago

changes z-index to 100 for draggable widgets

comment:2 jkudish2 years ago

  • Cc joachim.kudish@… added
  • Keywords has-patch added

comment:3 alexvorn22 years ago

for a better future support this can be changed to 1001 or 2001 z-index...

jkudish2 years ago

changes z-index to 1001 instead of 100

comment:4 follow-up: SergeyBiryukov22 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.5

We just need a bigger z-index than .wp-menu-arrow has (which is 25):
http://core.trac.wordpress.org/browser/tags/3.4/wp-admin/css/wp-admin.dev.css#L1477

20300-1.diff seems good.

Version 0, edited 22 months ago by SergeyBiryukov (next)

comment:5 in reply to: ↑ 4 helenyhou22 months ago

Replying to SergeyBiryukov:

20300-1.diff seems good.

Agreed. Let's not artificially inflate the z-index. :)

comment:6 jkudish22 months ago

  • Keywords 3.5-early added

comment:7 markjaquith22 months ago

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

In [21214]:

Increase the z-index of widgets so they don't appear behind menu portions while dragging. props jkudish. fixes #20300

Note: See TracTickets for help on using tickets.