Opened 2 years ago

Closed 2 years ago

Last modified 21 months ago

#17282 closed enhancement (fixed)

Dashboard Recent Comments "View All" replacement

Reported by: jane Owned by: ryan
Priority: normal Milestone: 3.2
Component: UI Version: 3.1
Severity: minor Keywords: has-patch
Cc: ryanimel

Description

The "View All" buttons on the dashboard boxes are weird b/c they are buttons, but they aren't actions, they are links. Would like to start switching over to a standard where buttons are mostly reserved for action/commitments, not just links to another screen.

Suggestions for Recent Comments box: instead of "View All" button, replace with links to the various views of the comments screen. So:

View All | Pending (24) | Approved | Spam (6) | Trash (3)

...where the link styles are the same as on edit-comments.php

Attachments (4)

dashboard.php.patch (180 bytes) - added by ryanimel 2 years ago.
Tweak of Dashboard Recent Comments widget
dashboard.dev.css.patch (77 bytes) - added by ryanimel 2 years ago.
Remove float from .subsubsub in widget
dashboard.css.patch (11.8 KB) - added by ryanimel 2 years ago.
dashboard_recent_comments.diff (877 bytes) - added by ryanimel 2 years ago.
This one should be correct. Sorry about the misstep.

Download all attachments as: .zip

Change History (15)

That would be the same as for the right now box, isn't it a repetition?

I believe the end goal would be to free up that space in the Right Now box for something else.

  • Cc ryanimel added

I don't actively use the Right Now box, so even if both boxes had these links it would be useful for me.

I put the list of views in the widget, removed the View All button. One thought I have, now that I see it there working: should the views be at the top of the Recent Comments box, rather than the bottom? Then again that might seem to indicate filtering would happen within the box itself, rather than linking off to another place.

Tweak of Dashboard Recent Comments widget

Remove float from .subsubsub in widget

comment:4 follow-up: ↓ 6   ryanimel2 years ago

  • Keywords has-patch added; needs-patch removed

comment:5   jane2 years ago

The goal is not to free up the Right Now box, the goal is to use buttons and links appropriately, and treat each box independently. A lot of people hide Right Now but keep Recent Comments. If we update Right Now and remove those items because they seem less important as standalone items there than they did in 2008 when we designed it, that would be a separate thing.

comment:6 in reply to: ↑ 4   azaozz2 years ago

Replying to ryanimel:

Hmm, seems something went wrong when creating or uploading the patches. Don't see anything in them. Also no need to do the minimized JS and CSS, the committer will do it when adding them. Please have a look here https://core.trac.wordpress.org/wiki#HowtoSubmitPatches

comment:7 follow-up: ↓ 8   markjaquith2 years ago

ryanimel — also, don't include minified files when making patches. Just patch the .dev.css or .dev.js versions. Less work for you, and less likely that they'll get out of sync if the committer is the one who minifies it.

comment:8 in reply to: ↑ 7   ryanimel2 years ago

Replying to azaozz:

Thanks, I figured I'd swing and miss here at first. New patch now, should be okay.

Replying to markjaquith:

That makes sense, will do.

This one should be correct. Sorry about the misstep.

comment:9   ryan2 years ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from new to closed

In [17797]:

Change View All Buttons into plain links. Show full set of status links in recent comments box. Drop Change Theme button.Props ryanimel. fixes #17282

Similar: #18306

Related: #17275

Note: See TracTickets for help on using tickets.