WordPress.org

Make WordPress Core

Opened 6 years ago

Last modified 13 months ago

#19711 assigned enhancement

Posts edit list window display no information about filtering

Reported by: yoavf Owned by: chriscct7
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Posts, Post Types Keywords: has-patch has-ui-feedback
Focuses: ui, administration Cc:

Description

The "All Posts" page lists the available posts.

It can filtered by authors or terms, for example by clicking on a category name or the author name next to a post.

The URL will change to apply the filtering, but there's no other visual indication of the filtering, nor a link to quickly disable filtering.

This can lead to confusion when users are accessing filtered lists directly or via links in other part of the dashboard.

Attachments (6)

19711.diff (1.4 KB) - added by sillybean 4 years ago.
First pass at adding filter indicator to post list page title
19711-author-dropdown-selection-box.diff (2.2 KB) - added by sworddance 4 years ago.
add author selection box to the post list page
19711.patch (2.0 KB) - added by chriscct7 2 years ago.
19711.2.patch (2.0 KB) - added by chriscct7 14 months ago.
19711.3.patch (2.0 KB) - added by chriscct7 14 months ago.
Changes to use post type label
19711.4.patch (2.0 KB) - added by chriscct7 14 months ago.
Fixes last patch. Also strtolower label name

Download all attachments as: .zip

Change History (26)

#1 in reply to: ↑ description ; follow-up: @DrewAPicture
6 years ago

  • Type changed from defect (bug) to enhancement

Replying to yoavf:

The URL will change to apply the filtering, but there's no other visual indication of the filtering, nor a link to quickly disable filtering.

This can lead to confusion when users are accessing filtered lists directly or via links in other part of the dashboard.

I agree. Maybe we could push the filter terms to an admin notice. Maybe something like:

Showing posts from: [Date], [Category]

#2 in reply to: ↑ 1 @helenyhou
6 years ago

Replying to DrewAPicture:

Maybe we could push the filter terms to an admin notice.

What about using the same placement as the message for search results? (span.subtitle within the h2)

#4 @helen
5 years ago

  • Component changed from UI to Administration
  • Keywords needs-patch ui-focus added; ui-feedback removed

@sillybean
4 years ago

First pass at adding filter indicator to post list page title

#5 @sillybean
4 years ago

  • Keywords has-patch ui-feedback added; needs-patch removed

@sworddance
4 years ago

add author selection box to the post list page

#6 @sworddance
4 years ago

sillybean's and (mine + enej)'s are complementary

#7 @sworddance
4 years ago

  • Cc sworddance added

#8 @jeremyfelt
4 years ago

  • Component changed from Administration to Posts, Post Types
  • Focuses admin added

@chriscct7
2 years ago

#9 @chriscct7
2 years ago

Refreshed both patches and combined them into a single patch

#10 @karmatosed
14 months ago

  • Keywords needs-testing added

@chriscct7 I was testing this and wanted to confirm what I should see different. If there could be some steps to test this that would be great. I did check the posts table but couldn't see anything visually changing.

There was an error running the patch so perhaps we just need a refresh? I want to run the patch to give some feedback UI wise.

Last edited 14 months ago by karmatosed (previous) (diff)

#11 @chriscct7
14 months ago

@karmatosed Patch indeed in need of refresh. Uploading refresh now.

With author (for example) selected:

Before:
Url: wp-admin/edit.php?post_type=post&author=1
Screenshot: http://screencloud.net/v/6Y0B

After:
Url: wp-admin/edit.php?post_type=post&author=1
Screenshot: http://screencloud.net/v/sK0s

@chriscct7
14 months ago

#12 @chriscct7
14 months ago

The output for the indicators is in the same place as where the output for the search term indicator goes for reference.

#13 @chriscct7
14 months ago

  • Keywords needs-testing removed
  • Milestone changed from Awaiting Review to 4.7
  • Owner set to chriscct7
  • Status changed from new to assigned

#14 @karmatosed
14 months ago

@chriscct7 thanks for that. I'm wondering a few things on looking at this:

  • Is there an existing style that works better for this? I ask as it feels a little stuck there. Maybe there is an existing pattern we can use that fits this more? I will note, after a tiny bit of looking one didn't strike me as better.
  • I like the visual feedback, it makes sense. I do wonder though if it's weird it appears and isn't there from the start.
  • On the pages list it says 'showing posts' - it should be pages I think there.

#15 @chriscct7
14 months ago

I agree it's a bit weird styling but that's what we currently style the search term feedback: http://screencloud.net/v/43j

Styling this + search term (since that will be there regardless of the merge of this or not) would probably be best handled on another ticket.

I like the visual feedback, it makes sense. I do wonder though if it's weird it appears and isn't there from the start.

Again, that's sorta how we do the search term thing. I think it makes enough sense that it appears and isn't really jarring. Also re:defaults this is again where we'd bump into the existing search functionality. A different ticket would probably better handle what to show if none of the filters or search is used OR if search isn't used, so that that can be done independent of this ticket.

On the pages list it says 'showing posts' - it should be pages I think there.

Updated!

@chriscct7
14 months ago

Changes to use post type label

#16 @chriscct7
14 months ago

Broke last patch. Reuploading

@chriscct7
14 months ago

Fixes last patch. Also strtolower label name

#17 @karmatosed
14 months ago

  • Keywords has-ui-feedback added; ui-feedback removed

@chriscct7 that seems to make sense to use the search format then, thanks for giving the UI context.

I found one thing that made me wonder in testing. I didn't see category showing but I do author:

https://cldup.com/nw4boMBUdE.gif

Is that a bug? I expected to see it even for 'Uncategorized'. I think if we fix that up then I would say the UI is ready to merge and thanks for keeping going with this one.

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


14 months ago

This ticket was mentioned in Slack in #core by desrosj. View the logs.


13 months ago

#20 @helen
13 months ago

  • Milestone changed from 4.7 to Future Release

Seems like not a lot of cases are covered and translations make me a little nervous - I don't think this requires an entire rethink of filtering UI and UX, but we need to do one of those anyway, so unless it's a small effort to make this better in the meantime, I think I'd rather tackle the bigger picture. In any case, not enough bandwidth for this before 4.7 beta.

Note: See TracTickets for help on using tickets.