-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Add micro delay to "Save Draft" button #4644
Comments
If we are adding a delay just to inform the user wouldn't working on messages help? I would suggest that from a design perspective for a critical feature like saving adding in delays could be not the best approach, or at least something to do after we look at the flow and how we can improve. |
There's some existing prior art for this in WordPress core but I can't remember the component.
Yes, worth revisiting the entire flow. |
Would be great to dig into that a bit and add context. |
@danielbachhuber did you find the prior art on this? |
I've looked twice now but can't find what I'm thinking of. @azaozz Do you recall what I'm referring to, or do I have a false memory?
|
Perhaps you mean the delay/disabling of the Save Draft button after clicking it to prevent double clicks and doing two identical requests? That's a standard UI feature that is needed everywhere :) With the addition of The difference is when the user saves a draft "manually", they also create a revision, autosaves don't create revisions, just update the post directly or may update the autosave revision if the post author is different than the current user. Also the client (i.e. Gutenberg) should probably do a "full save" for drafts from time to time instead of an autosave. This is a "weakness" in the classic editor/Edit Post screen; it would replace the whole post on autosaves and never create a draft, no matter what. However if the user did something by mistake, it will save that mistake with no way to recover from it. |
Closing as |
To improve the perceived UX of the "Save Draft" button, it would be helpful to add a micro delay to the transition:
From #1295 (comment)
The text was updated successfully, but these errors were encountered: