WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 6 months ago

Last modified 6 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:

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

Download all attachments as: .zip

Change History (22)

#1 @helen
2 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 months ago

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

@xavortm
8 months ago

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

@xavortm
8 months ago

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

@xavortm
8 months ago

The proper patch file

#5 @johnbillion
8 months ago

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

@xavortm
8 months ago

@xavortm
8 months ago

Screenshot of the end result

@xavortm
8 months ago

Minor tweaks for IE7/8 compability

#6 @xavortm
8 months ago

  • Keywords has-screenshots added; needs-screenshots removed

Minor tweak for IE7/8 compatibility

#7 @johnbillion
8 months ago

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

#8 @helen
8 months ago

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

@xavortm
8 months ago

#9 @xavortm
8 months ago

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

#10 @johnbillion
8 months ago

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

#11 @johnbillion
7 months ago

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

#12 @swissspidy
6 months ago

  • Keywords needs-testing added

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

#13 @johnbillion
6 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
6 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
6 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.