Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#21021 closed defect (bug) (fixed)

Unable to drag to empty second column on dashboard

Reported by: actionnerd's profile ActionNerd Owned by: azaozz's profile azaozz
Milestone: 3.4.1 Priority: normal
Severity: minor Version: 3.4
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

If, for whatever reason, the second column on the dashboard is empty it appears to be impossible to drag new panels over.

Tried it with the default theme installed by first dragging all the panels to the first column. Once they're all in the first column, can no longer add to second column.

Attachments (3)

21021.diff (463 bytes) - added by helenyhou 12 years ago.
21021.patch (455 bytes) - added by azaozz 12 years ago.
21021-2.patch (552 bytes) - added by azaozz 12 years ago.
Remove one more unneeded :empty

Download all attachments as: .zip

Change History (18)

#1 @nacin
12 years ago

Can't reproduce. What browser, OS?

#2 @ActionNerd
12 years ago

Chrome v19.0.1084.56
Firefox v13.0.1
OS X v10.7.4

I've made a small little movie file so you can see just in case: http://theexperiment.ca/files/dashboard.mov

#3 @knutsp
12 years ago

What plugins are active, if any? I can see a plugin update notice in the movie, but it might be deactivated, right?

#4 @ActionNerd
12 years ago

No active plugins.

Also a fresh install.

#5 @ericlewis
12 years ago

I reproduced on trunk with @ActionNerd's steps.

  1. On dashboard, move all widgets to left column.
  2. Try to move a widget to right column. Notice there will be no dashed option in the right column to move widget to.

#6 @ActionNerd
12 years ago

So I'm not crazy? Well that's a first. Thanks!

#7 follow-up: @nacin
12 years ago

  • Component changed from General to Administration
  • Milestone changed from Awaiting Review to 3.4.1

This is an issue with responsiveness.

At a certain width of the screen, widgets remain in the second column, but once removed, they can no longer go back. If you make your window wider, eventually the dashed option will show up.

It happens at 1200 pixels. But two columns of full widgets don't wrap to one column until 800 pixels. These two values should probably be the same, at 800 pixels.

@helenyhou
12 years ago

#8 @helenyhou
12 years ago

  • Keywords has-patch added

I feel like it can't possibly be this simple, but in my testing this worked: 21021.diff

#9 @nacin
12 years ago

  • Owner set to azaozz
  • Status changed from new to assigned

#10 in reply to: ↑ 7 @azaozz
12 years ago

Patch looks good. As far as I remember this line was there to prevent the border when auto-switching to one column and the second column is empty, but that's handled few lines above.

#11 @azaozz
12 years ago

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

In [21093]:

Fix dragging to empty second column on dashboard, props helenyhou, fixes #21021 for 3.4

#12 @azaozz
12 years ago

In [21094]:

Fix dragging to empty second column on dashboard, props helenyhou, fixes #21021 for trunk

#13 @azaozz
12 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Another related regression: IE8 doesn't like :empty and discards the other selector too. That results in not expanding the drop area, same as above.

@azaozz
12 years ago

@azaozz
12 years ago

Remove one more unneeded :empty

#14 @azaozz
12 years ago

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

In [21098]:

Fix dragging to empty second column in IE8, fixes #21021 for 3.4

#15 @azaozz
12 years ago

In [21099]:

Fix dragging to empty second column in IE8, fixes #21021 for trunk

Note: See TracTickets for help on using tickets.