WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years 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 3 years ago.
changes z-index to 100 for draggable widgets
20300-2.diff (441 bytes) - added by jkudish 3 years ago.
changes z-index to 1001 instead of 100

Download all attachments as: .zip

Change History (9)

comment:1 @helenyhou3 years ago

  • Component changed from Template to UI

@jkudish3 years ago

changes z-index to 100 for draggable widgets

comment:2 @jkudish3 years ago

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

comment:3 @alexvorn23 years ago

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

@jkudish3 years ago

changes z-index to 1001 instead of 100

comment:4 follow-up: @SergeyBiryukov3 years 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 3 years ago by SergeyBiryukov (next)

comment:5 in reply to: ↑ 4 @helenyhou3 years ago

Replying to SergeyBiryukov:

20300-1.diff seems good.

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

comment:6 @jkudish3 years ago

  • Keywords 3.5-early added

comment:7 @markjaquith3 years 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.