Make WordPress Core

Opened 12 years ago

Last modified 5 years ago

#19711 assigned enhancement

Posts edit list window display no information about filtering

Reported by: yoavf's profile yoavf Owned by: chriscct7's profile chriscct7
Milestone: 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 11 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 11 years ago.
add author selection box to the post list page
19711.patch (2.0 KB) - added by chriscct7 8 years ago.
19711.2.patch (2.0 KB) - added by chriscct7 7 years ago.
19711.3.patch (2.0 KB) - added by chriscct7 7 years ago.
Changes to use post type label
19711.4.patch (2.0 KB) - added by chriscct7 7 years ago.
Fixes last patch. Also strtolower label name

Download all attachments as: .zip

Change History (26)

#1 in reply to: ↑ description ; follow-up: @DrewAPicture
12 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
12 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
11 years ago

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

@sillybean
11 years ago

First pass at adding filter indicator to post list page title

#5 @sillybean
11 years ago

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

@sworddance
11 years ago

add author selection box to the post list page

#6 @sworddance
11 years ago

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

#7 @sworddance
11 years ago

  • Cc sworddance added

#8 @jeremyfelt
10 years ago

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

@chriscct7
8 years ago

#9 @chriscct7
8 years ago

Refreshed both patches and combined them into a single patch

#10 @karmatosed
7 years 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 7 years ago by karmatosed (previous) (diff)

#11 @chriscct7
7 years 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
7 years ago

#12 @chriscct7
7 years 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
7 years 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
7 years 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
7 years 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
7 years ago

Changes to use post type label

#16 @chriscct7
7 years ago

Broke last patch. Reuploading

@chriscct7
7 years ago

Fixes last patch. Also strtolower label name

#17 @karmatosed
7 years 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.


7 years ago

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


7 years ago

#20 @helen
7 years 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.