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

Indicating and resetting to default values #484

Open
nicolaskruchten opened this issue Jun 12, 2018 · 3 comments · May be fixed by #906
Open

Indicating and resetting to default values #484

nicolaskruchten opened this issue Jun 12, 2018 · 3 comments · May be fixed by #906

Comments

@nicolaskruchten
Copy link
Contributor

I promised @alexcjohnson that this editor would somehow indicate to the user when the value of an attribute has been explicitly set or is currently just following the underlying default, and that there would be a way to 'clear' the current setting. For example, if the color of a marker is blue, is it blue because the user set it to blue, or is blue the default? If it's the former, then we should be able to reset it to the default, even if the default is the same blue.

My current best-guess around what this could look like is this, with some more careful thought paid to the colours:

image

The dynamic UX would involve a tooltip on hover with a bit of explanatory text and the option to double-click to reset to defaults.

Thoughts @VeraZab ?

@alexcjohnson
Copy link
Collaborator

Nice, thanks @nicolaskruchten! I'm not sure about double-click though - If we're
going for a tooltip of some sort, why not let the bar expand into a single-clickable clear button on hover? Something like:
defaults

@nicolaskruchten
Copy link
Contributor Author

I'm a fan of that!

@VeraZab
Copy link
Contributor

VeraZab commented Jun 13, 2018

expanding on hover sounds good!

@dmt0 dmt0 self-assigned this Apr 11, 2019
@dmt0 dmt0 linked a pull request Apr 12, 2019 that will close this issue
3 tasks
@dmt0 dmt0 removed their assignment Feb 3, 2020
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

Successfully merging a pull request may close this issue.

4 participants