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

Consider to reset the sidebar panels state on the New Post page #2571

Closed
afercia opened this issue Aug 28, 2017 · 3 comments
Closed

Consider to reset the sidebar panels state on the New Post page #2571

afercia opened this issue Aug 28, 2017 · 3 comments
Labels
General Interface Parts of the UI which don't fall neatly under other labels. [Type] Enhancement A suggestion for improvement.

Comments

@afercia
Copy link
Contributor

afercia commented Aug 28, 2017

After #2533 the sidebar panels state is now saved in the browser's localStorage. That means, for example, if users expand the "Categories & Tags" panel (or any other panel), then switch to the Block setting, and then back again to the Post setting, they will find the panels in the same previous state.

Seems to me this is good when working on the same post, as it's the expected behavior.

However, there are scenarios where this may not be what users expect, for example:

  • edit a post, expand the "Categories & Tags" panel, do some edit and save
  • navigate away from the post editor
  • go on the "New Post" page
  • should users find the "Categories & Tags" panel expanded?

Maybe, in this case, the panels state should be reset to a sensible default (I guess with only the Status & Visibility panel open).

Not sure, but worth considering, if this applies also when navigating away from a post and then editing again the same post.

@afercia afercia added General Interface Parts of the UI which don't fall neatly under other labels. [Type] Enhancement A suggestion for improvement. labels Aug 28, 2017
@youknowriad
Copy link
Contributor

Well, this goes against #450 issue. Why do we bother to persist in localStorage, Redux state is enough in this case.

@youknowriad
Copy link
Contributor

@jasmussen Should we close this issue? Do we want the persistence across page reloads and new posts or just when navigating in the same post?

@jasmussen
Copy link
Contributor

I understand the use case, having a "fresh editor". But I do think it would be hard for a user to understand why sometimes the state is persisted, while other times it's not. As such, I do think that we should let the panels state be stored across edits and even posts.

I feel like this is something we can reopen if the use case becomes pressing in the future.

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. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

3 participants