WordPress.org
Get WordPress

Make WordPress Core

Opened 19 years ago

Closed 19 years ago

#5997 closed defect (bug) (fixed)

Grey line through Widget control div

Reported by: technosailor Owned by: mdawaffe
Priority: high Milestone: 2.5
Component: General Version: 2.5
Severity: normal Keywords: has-patch needs-testing
Cc: Focuses:

Description

When a widget control box is open, the grey horizontal separator lines leak through into the div. Being that it is such a light grey, it's hard to tell that it's a line and not a scratch on the monitor. Screenshot attached for Safari 3/OSX

Attachments (2)

Picture 1.png​ (32.0 KB ) - added by technosailor 19 years ago.
5997.diff​ (4.1 KB ) - added by mdawaffe 19 years ago.

Download all attachments as: .zip

Change History (8)

@technosailor
19 years ago

#1 @ryan
19 years ago

  • Owner changed from anonymous to mdawaffe

#2 @mdawaffe
19 years ago

  • Milestone 2.6 → 2.5
  • Priority normal → high

#3 @mdawaffe
19 years ago

  • Status new → assigned
  • Version → 2.5

So this was fairly simply fixed in Safari by tweaking some positioning.

It turns out things are pretty broken in IE, though.

Attached works in FF2, Safari3, IE7 (FF3, IE6 untested).

  1. Tweak some positioning.
  2. Bug fix for JS URL "unserialize" function.
  3. Some trivial validation fixes.
  4. If IE, don't animate widget open/close. Just do it. The animations seem to be breaking badly because of some positioning/floating.

If someone else can tweak the CSS and/or JS to make the animation work in FF, Safari and IE, please go ahead.

@mdawaffe
19 years ago

#4 @mdawaffe
19 years ago

  • Keywords has-patch needs-testing added

#5 @ryan
19 years ago

Works for me in FF2 and Safari3.

#6 @ryan
19 years ago

  • Resolution → fixed
  • Status assigned → closed

(In [7086]) Widget admin layout fixes from mdawaffe. fixes #5997

Note: See TracTickets for help on using tickets.