Show revision number for post/pages in Revision list

[midhund]

Sometimes the current revision title is not enough to identify the post/page revisions on the list. So I wish to show the revision number with link to revisions's page.

I have the patch, please review it.

Revision Number Patch #18733

Revision Number in Edit Page

Revision Number in Revision's Page

Revision Number in Revision's Page

This looks like a good idea.

Is this patch submission procedure is correct? missed anything?

The patch submission procedure is correct. Now you just need to get a commiter to commit it.

Before doing that though, please read this: ​http://wpdevel.wordpress.com/2011/09/22/freeze-just-in-time-for-fall/

I don't think the raw post ID is the best number to be presenting here. On many sites this ID will get into the hundreds, thousands, or larger, and they won't be sequential (or maybe not in the correct order, as the screenshot seems to indicate).

Looking at the DB, I don't see a sequential number we would want to use. Like revision 2, revision 3, etc.

@nacin : Yes, I know the raw post id is not sequential, but for the entire blog its should be unique (like this ticket id in Trac). And the user don't get confused when (suppose two posts have revision ids # 22) seeing # 22.
So what you think is the sequential number or unique in entire blog is more user friendly ?

@nacin: or else I can provide sequential number from the field name 'wp_posts.post_name'.
In post_type revision, the value is notated as

25-revision

25-revision-1

..

..

..

25-revision-5

25-revision-6

25-revision-7

for post id 25

for eg: by exploding the 25-revision-5 will get the sequence number of that revision as 5.

hope this makes sense.

Adding to the candidate list for the 3.6 Revisions feature work.

We should also consider the possibility of a revision change message when we integrate this extra info into the UI and how we might track, capture and display that.

Could you please describe 'revision change message'. Not much clear that.

this ties right into the ui refresh.

So there was a discussion with @nacin,

The revision id is unique in entire blog,
or
Unique in Post? then each post have revision 1, 2, 3, ...

Please find the attachments, I have some screens to show the revision. Showing the Deleted or Inserted characters doesn't make it simple for average users, (IMHO )I think those this are only for developers. So the user need to check his changed content not the number of characters, etc.[ ​http://make.wordpress.org/core/2013/01/07/wordpress-3-6-revisions/]

QucikRevision

revision meta quick reference

Replying to midhund:

Please find the attachments, I have some screens to show the revision. Showing the Deleted or Inserted characters doesn't make it simple for average users, (IMHO )I think those this are only for developers. So the user need to check his changed content not the number of characters, etc.[ ​http://make.wordpress.org/core/2013/01/07/wordpress-3-6-revisions/]

The revisions team are not sure that bringing all the diff details for each revision into the meta box makes sense. Instead we are focusing on improving the useful context info there which helps you get to the right revisions changes to review.

Replying to westi:

Replying to midhund:

Please find the attachments, I have some screens to show the revision. Showing the Deleted or Inserted characters doesn't make it simple for average users, (IMHO )I think those this are only for developers. So the user need to check his changed content not the number of characters, etc.[ ​http://make.wordpress.org/core/2013/01/07/wordpress-3-6-revisions/]

The revisions team are not sure that bringing all the diff details for each revision into the meta box makes sense. Instead we are focusing on improving the useful context info there which helps you get to the right revisions changes to review.

So this version numbering will be available on 3.6 release?

So, do we still need an ID here?

no, maybe if should have gravatar though, same as list above.

Replying to ocean90:

So, do we still need an ID here?

in 18733.diff​

• in post revision meta box ' restored from' line - add gravatar image, remove id, clean up language

before

after

clean up restored line in revisions post meta box, add gravatar

18733.2.diff​ fixes a notice in 18733.2.diff and removes the second arg from the human_time_diff() call to prevent a false output on other timezones.

IMO "Previously" should be kept.

We can also remove the translation context in line 33. It's not necessary here, it just partially duplicates the translator's comment.

In 24156:

Revisions: Remove the ID from the "restored by" line. props adamsilverstein. see #18733.

Moving this back to Future Release.

azaozz raised a comment on ​IRC which needs to be discussed also.