Ticket #4264 (closed defect: fixed)

Opened 1 year ago

Last modified 1 year ago

IE7, available widget drop zone won't expand past 2 rows of available widgets

Reported by: Denis-de-Bernardy Assigned to: andy
Priority: high Milestone: 2.2.1
Component: Administration Version: 2.2
Severity: major Keywords:
Cc:

Description

Per Matt's request:

http://trac.wordpress.org/ticket/4259#comment:1

Please see the attached screenshot

Attachments

widgets.png (67.7 kB) - added by Denis-de-Bernardy on 05/14/07 16:07:40.
widgets.diff (2.3 kB) - added by Denis-de-Bernardy on 05/18/07 21:46:32.
placematt.png (19.3 kB) - added by Denis-de-Bernardy on 05/19/07 01:50:12.
reason why placematt class should get dropped
4264.RTL.diff (1.0 kB) - added by Sewar on 07/19/07 10:50:55.
RTL support

Change History

05/14/07 16:07:40 changed by Denis-de-Bernardy

  • attachment widgets.png added.

05/14/07 19:37:05 changed by Otto42

  • version set to 2.1.3.

Confirmed. Same issue exists with: -Latest version of the Sidebar Widgets plugin, Wordpress 2.1.3, and IE7. -Wordpress.com and IE7

Have not confirmed issue with widgets in 2.2 yet.

The "default sidebar" things seem to be centered on the page in IE7 for some reason.

05/14/07 20:12:51 changed by rob1n

  • version changed from 2.1.3 to 2.2.

I'm pretty sure he means the integrated widgets in 2.2.

05/15/07 22:15:59 changed by Denis-de-Bernardy

It's in the widgets plugin and the integrated widgets in WP 2.2 and (last I checked) the widgets used on wordpress.com.

This really is a blocker imho.

05/15/07 22:26:11 changed by rob1n

It just looks different, right? Still functional?

05/15/07 22:33:12 changed by ryan

I agree that it is ugly and needs to be fixed, but I don't think it is a blocker.

05/15/07 22:35:54 changed by rob1n

I think this can be staved off for 2.2.1.

05/15/07 22:36:02 changed by rob1n

  • owner changed from anonymous to andy.

05/15/07 22:39:48 changed by rob1n

  • milestone changed from 2.2 to 2.2.1.

05/15/07 22:43:43 changed by intoxination

There is more than just appearance, although minor. In IE7, the available widget drop zone won't expand past 2 rows of available widgets. That prevents you from dragging all the widgets back down to that area if you want (one you get your 2 rows, you can't add anymore - they just float back to the top).

Again - it is something that probably won't affect many people, but it is a functionality problem.

05/15/07 22:47:39 changed by technosailor

I don't use IE so missed this, but I tend to agree that it needs to be fixed prior to 2.2 public download. I know we won't push 2.2 to our blogs until 2.2.1 if it is not addressed because we do use widgets ALOT and too many (for my liking) of our bloggers use IE.

05/15/07 22:50:57 changed by foolswisdom

  • summary changed from widgets very broken in IE to IE7, available widget drop zone won't expand past 2 rows of available widgets.

05/16/07 00:37:15 changed by Denis-de-Bernardy

  • milestone changed from 2.2.1 to 2.2.

Rob1n & Ryan: It's not functional. See comment by intoxination, 05/15/07 22:43:43. It's true annoyance for IE7 users as soon as they've more than a few plugins in the admin interface. If I recall correctly, they can no longer remove widgets past a certain point.

D.

05/16/07 02:20:03 changed by JeremyVisser

  • milestone changed from 2.2 to 2.2.1.

2.2 released

05/17/07 00:44:32 changed by Denis-de-Bernardy

For information, I've placed a bid request on rentacoder for this one:

http://www.rentacoder.com/RentACoder/misc/BidRequests/ShowBidRequest.asp?lngBidRequestId=687366

D.

05/18/07 21:46:32 changed by Denis-de-Bernardy

  • attachment widgets.diff added.

05/18/07 21:51:59 changed by Denis-de-Bernardy

The attached patch makes things work as expected in FF2 and IE7. Can anyone with IE6 and other browsers give it some love?

D

05/19/07 00:34:42 changed by ryan

The part of the patch that changes "placematt" to "placemat" didn't apply cleanly so I left it out. Seemed to behave well with IE6 running in wine on Ubuntu.

05/19/07 00:37:35 changed by ryan

Forced it to expand to 3, 4, and then 5 rows of widgets. Looked good on IE6 and FF2/linux. Elements within the sidebars lined up properly.

05/19/07 01:45:22 changed by Denis-de-Bernardy

actually, the part with the 'placematt' changed to 'placemat' does apply -- it could be cleaned up by dropping the placematt class entirely.

If it stays around, it's a big mess with themes with three columns (I'll attach a screenshot).

05/19/07 01:50:12 changed by Denis-de-Bernardy

  • attachment placematt.png added.

reason why placematt class should get dropped

05/19/07 17:23:50 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

(In [5493]) Fix widget admin layout for IE7. Props Denis-de-Bernardy. fixes #4264 for 2.2

05/19/07 17:24:25 changed by ryan

(In [5494]) Fix widget admin layout for IE7. Props Denis-de-Bernardy. fixes #4264 for 2.3

05/19/07 17:29:24 changed by ryan

I put this live on wordpress.com for further testing and feedback. Looking good so far.

07/19/07 10:50:55 changed by Sewar

  • attachment 4264.RTL.diff added.

RTL support

07/19/07 10:52:42 changed by Sewar

  • status changed from closed to reopened.
  • resolution deleted.

Let's fix RTL also.

07/19/07 17:42:27 changed by foolswisdom

  • status changed from reopened to closed.
  • resolution set to fixed.

Sewar? A ticket with a released milestone? Please open a new ticket referencing this one.