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

Add obvious save button for renaming devices #5012

Closed
turt2live opened this issue Sep 11, 2017 · 7 comments
Closed

Add obvious save button for renaming devices #5012

turt2live opened this issue Sep 11, 2017 · 7 comments
Labels
P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Design

Comments

@turt2live
Copy link
Member

No description provided.

@lampholder
Copy link
Member

I didn't even realise you could rename the devices using the Device list in user settings :\

There's a lot wrong with this page, including that all (I think all) changes are instantly persisted (in a way that is entirely different to how room settings are persisted).

If we did want to address this specific problem in advance of a more comprehensive rework of settings, I might suggest something where:

  • editability is demonstrated on hover
  • clicking to edit actually opens a modal
  • the modal gives the option to save/cancel

I'd be afraid though that adding this kind of save confirmation to just this aspect of the user settings might create confusion about how the other user settings are saved, so it's probably worth tackling the problem more holistically.

I made a quick mockup to help me think through the issues, which I've attached here for posterity.
image

@lampholder lampholder added T-Defect design S-Minor Impairs non-critical functionality or suitable workarounds exist P2 labels Sep 12, 2017
@turt2live
Copy link
Member Author

The mockup overall looks good, although I'd personally be annoyed with a dialog. Instead, can the field change to just an input element with save and cancel icons on the right side?

@t3chguy
Copy link
Member

t3chguy commented Sep 12, 2017

isn't that section already struggling for space on low width displays?

@lampholder
Copy link
Member

That's what I thought - a dialog is annoying, but it's not a super common operation.

Something nicer would be something that replaced some of the junk you don't need to look at whilst editing (timestamp, ip) with some save/cancel options, but I'd rather not introduce a radical new settings modfication paradigm without thinking how it would apply to other bits of the UX :)

@turt2live
Copy link
Member Author

/me quietly recommends the in-place edit paradigm everywhere ;)

@lampholder
Copy link
Member

I think I agree, but I want to achieve emphatic consensus before committing to:

  • doing that for all new UX
  • opportunistically introducing it wherever we touch old UX

😄

@turt2live
Copy link
Member Author

we did this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Design
Projects
None yet
Development

No branches or pull requests

4 participants