Dashboard Recent Comments "View All" replacement

[jane]

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

[ocean90]

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

[nacin]

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

[ryanimel]

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.

[ryanimel]

Tweak of Dashboard Recent Comments widget

[ryanimel]

Remove float from .subsubsub in widget

[jane]

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.

[azaozz]

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

[markjaquith]

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.

[ryanimel]

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.

[ryanimel]

This one should be correct. Sorry about the misstep.

[ryan]

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

[scribu]

Similar: #18306

[ramiy]

Related: #17275