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

[dashboard] Make functionality of the dotfiles settings button more clear #7639

Closed
filiptronicek opened this issue Jan 17, 2022 · 5 comments · Fixed by #10136
Closed

[dashboard] Make functionality of the dotfiles settings button more clear #7639

filiptronicek opened this issue Jan 17, 2022 · 5 comments · Fixed by #10136

Comments

@filiptronicek
Copy link
Member

Is your feature request related to a problem? Please describe

In the user preferences you can now set a repository for your dotfiles, which is great, but when you put in your desired dotfiles repository and you click Save Changes, it makes you question if you need to use this button for all settings (it really doesn't, I checked). It is the only setting that does not autosave after the user makes a selection and that makes it look like the button below it should be used for all preference changes the user initiates.

Describe the behavior you'd like

It would probably make sense to separate the different sections of the preferences page (maybe add some horizontal rules) and/or change the text of the button to something more indicative of the setting which is actually saved or maybe even borrow something from GitHub's repository settings and add make the button appear a horizontal flexbox with the text input:
image

Describe alternatives you've considered

Autosave, but that would not be ideal, ever, maybe only after an onpaste event.

@gtsiolis
Copy link
Contributor

Thanks for noticing, @filiptronicek! Improving the dotfiles repository selection is something we also track in #7385.

However, I agree that moving that Save Changes button next to the text input element as an inline form could help avoid some confusion as users could assume that button is required for persisting all changes inside the preferences page.

Marking this also as a good-first-issue.

🍊 🍊 🍊 🍊

An additional good intermediate step could be to either introduce a better user feedback upon save as described in #7337 (comment):

Auto-saving the input here happens in the background but is lacking 🅰️ call to action and 🅱️ some user feedback upon save which both will probably be resolved in the second iteration in #7385.

For 🅰️, we could add an action button below to save changes so that users won't be surprized when they accidentally change this input and dotfiles configuration fails, etc. Alternatively, we could introduce a new auto-save pattern that makes it visible to the user that we auto-save on every key press.

For 🅱️, ideally this manual save action would also need some feedback in the user interface so that users can safely acknowledge that the setting has been updated. This could be done with a subtle loading indicator (spinner) inside the button when saving changes that remains visible for a noticeable period of time or using a toast notification (#3530) which can be considered out of the scope of this PR.

@gtsiolis gtsiolis added team: webapp Issue belongs to the WebApp team user experience labels Jan 25, 2022
@sagor999
Copy link
Contributor

👍 on this issue.
Some sort of feedback that settings were saved would be helpful too. Right now when pressing the button "save changes" it appears like nothing happens.

@andreafalzetti
Copy link
Contributor

I was about to create an issue for this, but it already exists, so +1 from me too :)

@loujaybee
Copy link
Member

This is not just an issue with dotfiles, but when adjusting the IDE preferences (which is auto-saved) this button also appears to be necessary (as it's at the end of the form, when in fact it's unrelated to the IDE preferences)

@gtsiolis
Copy link
Contributor

Thanks @loujaybee, @andreafalzetti, @sagor999, and @filiptronicek for the feedback! Could not agree more with all the points you listed.

In the spirit of minimum viable changes and shipping, I've opened #10136 to improve the current state until we pick up #7385 which will take into account improvement on the new workspace input modal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants