Screen Options calls all post types "posts"

[caesarsgrunt]

The option "Posts per page" under Screen Options on edit.php should change to reflect the page type - eg "Books per page" for a custom post type of Book.

Might be a problem with "Pages" - Pages per page is a little confusing".
One solution would be to make it something like "{{post_type_label}} per list" or "Show [input] {{post_type_label}} at once".
An alternative would be to make it "Items per page", but I don't really like that.

[caesarsgrunt]

Or even just make it "Show [input] {{post_type_label}}", without mentioning that it's per page or whatever.

[caesarsgrunt]

Or perhaps the pest answer is "{{post_type_label}} per screen", seeing as we already have the label "screen options" etc.

[nacin]

To me, "per screen" sounds better than "per page". Could use some UX feedback on this one.

[uglyrobot]

I agree with per screen.

Also the post updated notice (yellow box) on the post.php screen says "Post updated. View post" on custom post types. It needs to be updated. The view button next to permalinks shows the custom post type name correctly.

[caesarsgrunt]

This first patch shows the correct post type label.

It also changes all instances of "{{Items}} per page" to "{{Items}} per screen" so that we don't end up with "Pages per page", as well as for consistency with other places where "screen" is used instead of "page".

I'll look at the yellow notice issue in a separate patch.

[nacin]

"per screen" has UX sign-off. Patch looks good, will kick the tires on it and check it in soon when I have a chance.

[jane]

I like using the word screen instead of page in general, but when we're talking specifically about pagination, which is what this is, page is probably more appropriate. "Items per page" would be my suggestion (and was, via IRC, before reading this thread, ha).

[caesarsgrunt]

Replying to jane:

I like using the word screen instead of page in general, but when we're talking specifically about pagination, which is what this is, page is probably more appropriate. "Items per page" would be my suggestion (and was, via IRC, before reading this thread, ha).

Thanks for weighing in, Jane.

I personally feel that "{{post type}} per screen" is more in keeping with language elsewhere in the admin, such as "Screen Options", "Show on screen", etc, and I also think that it could cause a certain amount of confusion with Pages as a post type.

However, your suggestion of "Items per page" does have the advantage of being more similar to the existing label, which it's true doesn't seem to cause much confusion, so I am happy with it as a second-best option.

So long as one or other gets committed I'll be happy!

[caesarsgrunt]

Alternatively, what do you think about "Show [input] {{post_type_label}}", without mentioning that it's per page?

[jane]

Screen refers to everything within the browser, while items-per refers to paginated views, so page is more correct than screen. "Show [post-type]s" would be better stated as "List [post-type]s" based on how we use Show currently in the media uploader. Not that I like the sound of it.

Side note: I just went and looked at Screen Options as it is in 2.9. I thought we'd agreed to get rid of that "Options" header above the #-per-page back in 2.9 (the whole thing is for screen options, and the # is for displaying on screen, after all (which is the label for the upper section). When this ticket gets done one way or another, could the person making the patch also get rid of that Options header? It almost makes it look like you can decide how many posts per page to show on your blog or something.

If that gets fixed, then whatever the #-per-page language is, it will be the completion of the header, "Show on screen," and just having "_ [post-type]s" would be adequate. Alternately, "_ [post-type]s at a time" would work.

[jane]

Forgot how trac autoformats with underlines, not my intention. Restating underlined parts to be more understandable.

Replying to jane:

Screen refers to everything within the browser, while items-per refers to paginated views, so page is more correct than screen. "Show # [post-type]s" would be better stated as "List # [post-type]s" based on how we use Show currently in the media uploader. Not that I like the sound of it.

Side note: I just went and looked at Screen Options as it is in 2.9. I thought we'd agreed to get rid of that "Options" header above the #-per-page back in 2.9 (the whole thing is for screen options, and the # is for displaying on screen, after all (which is the label for the upper section). When this ticket gets done one way or another, could the person making the patch also get rid of that Options header? It almost makes it look like you can decide how many posts per page to show on your blog or something.

If that gets fixed, then whatever the #-per-page language is, it will be the completion of the header, "Show on screen," and just having "# [post-type]s" would be adequate. Alternately, "# [post-type]s at a time" would work.

[caesarsgrunt]

If I understand right, you're saying

1. Remove the "Options" header
   
2. Change the "# posts per page" to "# [post_type]" or "# [post_type] at a time".

Sounds good to me.

Which string do you prefer? With or without at a time?

[jane]

@caesarsgrunt: Yes, you understand correctly. Part of me likes "at at time", but the part of me that thinks about translation thinks just "# [post-type]s" is probably better, so I'd say no 'at a time'.

[caesarsgrunt]

I speak multiple languages and can't see a problem re translation with "at a time" - eg Spanish, "# [post-type] a la vez" - but okay, I'll do it without.

[caesarsgrunt]

Change label to "# [post_type]"

[caesarsgrunt]

There it is without "at a time". It's trivial enough to change that I'll upload it with as well, then you can commit whichever you like.

[caesarsgrunt]

Change label to "# [post_type] at a time"

[caesarsgrunt]

Patch 3 is with "at a time". Rather than add it to each string I've made "%s at a time" a separate string for translation, which I think is better than the previous system of having a load of individual but similar strings.

[nacin]

I'd prefer just "[ # ] Posts", though, what about "Show [ # ] Posts"?

[nacin]

(In [14157]) Change the 'per page' screen options strings. fixes #12872, props caesarsgrunt.