Make WordPress Core

Opened 11 years ago

Closed 9 years ago

Last modified 9 years ago

#26399 closed enhancement (fixed)

Improve Handling of "Blank" Dashboard Columns

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

Description

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 9 years ago.
26399-2.diff (2.0 KB) - added by xavortm 9 years ago.
26399-3.diff (2.5 KB) - added by xavortm 9 years ago.
The proper patch file
26399.2.diff (2.7 KB) - added by xavortm 9 years ago.
2015-10-25_154615.png (33.9 KB) - added by xavortm 9 years ago.
Screenshot of the end result
26399.3.diff (3.0 KB) - added by xavortm 9 years ago.
Minor tweaks for IE7/8 compability
26399.4.diff (3.0 KB) - added by xavortm 9 years ago.

Download all attachments as: .zip

Change History (22)

#1 @helen
11 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
9 years ago

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

@xavortm
9 years ago

#3 @xavortm
9 years 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.

@xavortm
9 years ago

#4 @xavortm
9 years 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 ;/

@xavortm
9 years ago

The proper patch file

#5 @johnbillion
9 years ago

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

@xavortm
9 years ago

@xavortm
9 years ago

Screenshot of the end result

@xavortm
9 years ago

Minor tweaks for IE7/8 compability

#6 @xavortm
9 years ago

  • Keywords has-screenshots added; needs-screenshots removed

Minor tweak for IE7/8 compatibility

#7 @johnbillion
9 years ago

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

#8 @helen
9 years ago

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

@xavortm
9 years ago

#9 @xavortm
9 years ago

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

#10 @johnbillion
9 years ago

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

#11 @johnbillion
9 years ago

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

#12 @swissspidy
9 years ago

  • Keywords needs-testing added

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

#13 @johnbillion
9 years 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
9 years 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
9 years ago

In 36296:

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

See #26399.

Note: See TracTickets for help on using tickets.