Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unified publish flow: return revisions to more prominence #62041

Closed
annezazu opened this issue May 28, 2024 · 8 comments · Fixed by #62323
Closed

Unified publish flow: return revisions to more prominence #62041

annezazu opened this issue May 28, 2024 · 8 comments · Fixed by #62323
Labels
Needs Design Feedback Needs general design feedback. [Package] Edit Post /packages/edit-post [Type] Feedback Issues that relate purely to feedback on a feature that isn't necessarily actionable

Comments

@annezazu
Copy link
Contributor

In testing 6.6, I noticed that revisions is now a bit more hidden and doesn't include the number of revisions available after this PR was merged: #61867

Before:

before.mov

After:

after.revisions.mov

In talking with enterprise folks, I imagine this might cause some panic and confusion. Revisions are a key part of the foundational experience of what WordPress offers and, when many folks are working on a piece, having easy access is important. Opening this issue partially to see if feedback comes up within the 6.6 cycle and to raise an initial flag.

@annezazu annezazu added Needs Design Feedback Needs general design feedback. [Type] Feedback Issues that relate purely to feedback on a feature that isn't necessarily actionable [Package] Edit Post /packages/edit-post labels May 28, 2024
@ntsekouras
Copy link
Contributor

ntsekouras commented May 28, 2024

Thanks for raising this Anne! I just wanted to mention that the revisions number is going to be part of the label here. We're having some discussion about the API, but it will be fixed for 6.6.

--edit

The PR that adds the revisions count was merged.

@annezazu
Copy link
Contributor Author

Oh amazing! I think that'll be important to have.

@aaronjorbin
Copy link
Member

I don't know if displaying the number of revisions is nearly as important as the findability of revisions from within the editor.

I also don't know if dataviews is enough of a place for the revisions since 1) It's currently limited to pages out of the box 2) it's not the default experience for navigating into the post editor.

Before #61867, to know if a post has any revisions, you just needed to have the document inspector open. Now you need to click. So that process goes from zero clicks to one. If the goal is to reduce the prominence, putting it inline with the last edited timestamp might be worth exploring.

Screenshot 2024-05-31 at 3 07 32 PM
@ppolo99
Copy link

ppolo99 commented Jun 3, 2024

Before #61867, to know if a post has any revisions, you just needed to have the document inspector open. Now you need to click. So that process goes from zero clicks to one. If the goal is to reduce the prominence, putting it inline with the last edited timestamp might be worth exploring.

I fully agree with this. Revisions are a key part of editorial teams as Anne mentions, and hiding them behind a menu doesn't make sense to me. I like the idea of having a timestamp or even a revisions icon in place of the text (Google Docs).

@smithjw1
Copy link

smithjw1 commented Jun 3, 2024

The original issue says:

Revisions are now accessible from the ellipsis menu which offers more appropriate prominence.

But I don't see any data or reasoning to support the second half of that statement. Do we know how often Revisions are accessed across sites? Does that vary by the average number of revisions on a post? How are we determining appropriate prominence?

With a shared understanding, we can have a more inclusive conversation about the proposed change vs. the original vs. @aaronjorbin's suggestion that strikes a middle ground.

@hrkhal
Copy link

hrkhal commented Jun 4, 2024

What is the onus for moving the revisions out of sight? As @annezazu has already noted this isn't a good move for enterprise users. Information like this should remain visible for the editors at a glance.

6.5 already brought in post url truncation in the editor, which again limits the information available for editors and users without interaction.

It now requires another click to see the full url for the post, if you add in the two clicks to see if there are any revisions and that's 3 extra clicks per editor per article per day. It adds up quick in an enterprise setting.

@annezazu
Copy link
Contributor Author

annezazu commented Jun 5, 2024

Thanks, folks, for sharing broader context and thoughts to round out my initial instincts on the enterprise side. A PR is in progress to explore making revisions more prominent again. You can use this Playground link to test it.

@annezazu
Copy link
Contributor Author

Thanks, Nik, for working on this and thank you everyone else for chiming in with feedback! Final state is a returned section in the sidebar like so:

Screenshot 2024-06-09 at 10 53 08 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Feedback Needs general design feedback. [Package] Edit Post /packages/edit-post [Type] Feedback Issues that relate purely to feedback on a feature that isn't necessarily actionable
6 participants