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

UI: Separate out drafting, saving, publishing actions #707

Closed
jasmussen opened this issue May 8, 2017 · 7 comments · Fixed by #1301
Closed

UI: Separate out drafting, saving, publishing actions #707

jasmussen opened this issue May 8, 2017 · 7 comments · Fixed by #1301
Assignees
Labels
General Interface Parts of the UI which don't fall neatly under other labels.
Milestone

Comments

@jasmussen
Copy link
Contributor

jasmussen commented May 8, 2017

We'll soon be able to save and publish posts! #594!

Once that's in, we should consider splitting the flow up, so the publish button doesn't do it all.

Saving & Publishing

saving publishing

The indicator on the left shows the saved state. It autosaves the post, like it always has, and indicates when changes are saved.

When there are unsaved changes, you can either wait for the auto-save to kick in, or you can click "Save", as a blue button.

The publish button has two roles:

  • Publishing the post the first time
  • Updating a published post

In addition to this, it can show a "working" animation when content is being published or updated.

Edit: I updated the mockup to remove the dropdown button, as that is unrelated to this ticket. See also #707 (comment).

@jasmussen jasmussen added General Interface Parts of the UI which don't fall neatly under other labels. Design labels May 8, 2017
@jasmussen jasmussen added this to the Beta milestone May 10, 2017
@jasmussen
Copy link
Contributor Author

Per discussion in #708 (comment) we should also have a "Schedule" state for the button.

@youknowriad youknowriad self-assigned this May 30, 2017
@youknowriad
Copy link
Contributor

So the button text could be either "Publish", "Update" or "Schedule".

Currently, we also have "Saving", "Updating", "Saved", "Updated", "Save failed", "Update failed". We should drop this status from the button text itself, right!?

@jasmussen
Copy link
Contributor Author

Currently, we also have "Saving", "Updating", "Saved", "Updated", "Save failed", "Update failed". We should drop this status from the button text itself, right!?

Right. Saving and Updating should probably both be the same, "Saving..." and sit in the area on the left

"Published", "Save failed", "Update failed", "Updated" is something that might actually require a notice, like we have in Calypso. Maybe I need to open a new ticket for notices with mockups. But we don't want them in the button, for sure.

@nylen
Copy link
Member

nylen commented May 30, 2017

Does this design require two clicks (one to open the dropdown, and the other to click the "real" button) to update a post? That seems undesirable to me.

@jasmussen
Copy link
Contributor Author

Does this design require two clicks (one to open the dropdown, and the other to click the "real" button) to update a post? That seems undesirable to me.

The dropdown isn't part of this ticket, sorry, mockup is misleading. But yes, #708, suggests an extra click. Note: that extra click is for these actions:

  • Publishing
  • Scheduling a post to be published
  • Updating a published post

It is not for saving a draft (which is done on the left side in this ticket), or for picking the schedule date, which is done in the sidebar, or could be done in the dropdown itself.

There are a couple of benefits to this:

a) it puts a barrier between you and the published post, an implicit "yes, I'm sure I want to publish this post". It's a replacement for the "confirm publish" alert, which has been requested a number of times on WordPress.com, and is also being discussed in https://core.trac.wordpress.org/ticket/39841
b) the dropdown itself is a chance to show the most important metadata, at the very last minute — like "did you remember to tag?"
c) it allows us an affordance to let you add that metadata without actually using the sidebar, essentially letting users who don't need the vast majority of metaboxes never open the sidebar after dismissing it the first time

I'm a believer in this design. I think it's good for everyone, and so long as the save action isn't buried there, it's desired friction.

That being said, it is a separate ticket from this one, because it is not as important to get to. Or to put it differently: if we don't get to it at all, that's okay too.

@youknowriad
Copy link
Contributor

closed by #945

@jasmussen
Copy link
Contributor Author

There seems to have been a regression here, as discovered in #1295. I can no longer get the "Saving..." state to invoke. Is this tracked somewhere else?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Interface Parts of the UI which don't fall neatly under other labels.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants