Make WordPress Core

Opened 4 years ago

Closed 19 months ago

Last modified 19 months ago

#26399 closed enhancement (fixed)

Improve Handling of "Blank" Dashboard Columns

Reported by: kraftbj Owned by: johnbillion
Milestone: 4.5 Priority: normal
Severity: normal Version: 3.8
Component: Administration Keywords: has-patch has-screenshots commit
Focuses: administration Cc:


With the new Dash improvements, on wide monitors, four columns exist for dashboard widgets.

If the user is not using all visible, a blank, dotted-borded box is displayed.

This can be confusing to users who don't realize they can drag the widget to those spaces, thinking the dotted lines are placeholders for future widgets.

Ideas for improvements:

  • Change the dotted box to some type of note that widgets can now be moved there.
  • If not being used, show a smaller blank indicator to maximize the size of the columns being used (A compromise for those who don't want 3 or 4 columns while keeping our fully responsive goal).

Attachments (7)

26399.diff (1.8 KB) - added by xavortm 21 months ago.
26399-2.diff (2.0 KB) - added by xavortm 21 months ago.
26399-3.diff (2.5 KB) - added by xavortm 21 months ago.
The proper patch file
26399.2.diff (2.7 KB) - added by xavortm 21 months ago.
2015-10-25_154615.png (33.9 KB) - added by xavortm 21 months ago.
Screenshot of the end result
26399.3.diff (3.0 KB) - added by xavortm 21 months ago.
Minor tweaks for IE7/8 compability
26399.4.diff (3.0 KB) - added by xavortm 21 months ago.

Download all attachments as: .zip

Change History (22)

#1 @helen
3 years ago

I agree that this still feels wrong, even though it's been that dotted outline for a few releases now. I also mentioned elsewhere that it could be nice to be smarter about how many columns there are depending on how many widgets are showing - trying to enforce 4 columns when only 2 widgets are even available is pretty lame.

#2 @chriscct7
22 months ago

  • Focuses administration added
  • Keywords needs-patch ux-feedback added

21 months ago

#3 @xavortm
21 months ago

I added the message "Drag panels here". Please keep in mind its my first ever patch ever, so i any feedback on it would be helpful.

21 months ago

#4 @xavortm
21 months ago

After having those two changes the string now will be translatable. No DOM elements are being added + it will work on any other empty-container elements. @johnbillion checked it live here at WordCamp Sofia on the contributor's day :) I am not 100% sure if the patch file i attached is properly set, since the first and second one must be applied ;/

21 months ago

The proper patch file

#5 @johnbillion
21 months ago

  • Keywords has-patch needs-screenshots added; needs-patch removed

21 months ago

21 months ago

Screenshot of the end result

21 months ago

Minor tweaks for IE7/8 compability

#6 @xavortm
21 months ago

  • Keywords has-screenshots added; needs-screenshots removed

Minor tweak for IE7/8 compatibility

#7 @johnbillion
21 months ago

  • Owner set to johnbillion
  • Status changed from new to reviewing

#8 @helen
21 months ago

Just a note that they're referred to as "boxes" in help text, not panels.

21 months ago

#9 @xavortm
21 months ago

From the last patch - updated to "Drag boxes here". Thanks for the note :)

#10 @johnbillion
21 months ago

  • Keywords 4.5-early added; ux-feedback removed
  • Milestone changed from Awaiting Review to Future Release

#11 @johnbillion
20 months ago

  • Keywords 4.5-early removed
  • Milestone changed from Future Release to 4.5

#12 @swissspidy
19 months ago

  • Keywords needs-testing added

Patch still applies and works as expected in Chrome. Needs some testing in other browsers.

#13 @johnbillion
19 months ago

  • Keywords commit added; needs-testing removed

Tested visually and functionally in Firefox 41, Safari 8, and Chrome 47, IE8, and IE9. IE8 does not show the new message, but remains functional as it is currently.

#14 @johnbillion
19 months ago

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

In 36295:

Administration: Add a "Drag boxes here" message to empty dashboard meta boxes so it's clear to users that it's possible to drag meta boxes into empty spaces.

Props xavortm
Fixes #26399

#15 @swissspidy
19 months ago

In 36296:

Administration: After [36295], mark postBoxL10n as a global to please JSHint.

See #26399.

Note: See TracTickets for help on using tickets.