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

Remove "exit workspace" part of the Save workspace button action #167

Open
ggeisler opened this issue Jun 15, 2021 · 1 comment
Open

Remove "exit workspace" part of the Save workspace button action #167

ggeisler opened this issue Jun 15, 2021 · 1 comment

Comments

@ggeisler
Copy link
Contributor

I'm a little on the fence with this one so if anyone has thoughts contrary to what I'm proposing, feel free to say so.

Currently when the user selects the "Save workspace" action (when working in a Mirador workspace) that action does two things (from the user perspective):

  • Saves the current workspace state (snapshot)
  • Exits the workspace and redirects the user to the details page of the workspace

I think we should only do the first of those -- update the workspace snapshot -- but not the second. We should remain in the workspace.

My concern is that especially when doing real annotation editing, users are likely to select the "Save workspace" button at times when all they intend to do is the labelled action: save the workspace. Based on how many software applications work, when users see a "Save workspace" button they are likely to use it just to make sure they don't lose work they've been doing (even though in Mise they are not really in danger of losing annotation editing work that way). And if they're not really done with their workspace work, it's pretty annoying to find yourself on the Workspace details page and having to open the workspace again.

The tradeoff is sometimes a user will want to do what the current action does do: save the workspace state and exit the workspace. In that case, we'd be requiring an extra action, since they'd have to select the "Back" action after the "Save workspace" action. But that feels to me like a more expected flow than the "Save workspace" button doing both things at once.

@mjgiarlo mjgiarlo self-assigned this Jun 17, 2021
@mjgiarlo mjgiarlo removed their assignment Jul 12, 2021
@mjgiarlo
Copy link
Member

Unassigning for now as I'm focused on hiring atm.

mjgiarlo added a commit that referenced this issue Jul 12, 2021
Connect to #167

This is a work-in-progress commit, as I have now unassigned from #167. Feel free to use this or ignore it or dump it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants