-
Notifications
You must be signed in to change notification settings - Fork 35
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
Selecting a visualization for a column should not require clicking the check box #457
Comments
We unified all dialogs and added the live preview feature with PR #265. Hence, you need to confirm the changes with the tickmark at the bottom. If we want to change this behavior (again), we should asses it for all dialogs to avoid specific dialogs behaviors again. |
I think this is all fine, but I do think that the default should be accept, not reject the changes. Or why should it not be like that? Can you give me an example where this is odd? And how is reset useful? |
Yes, makes sense. Should be just changing the default value here: lineupjs/src/ui/dialogs/ADialog.ts Line 238 in 5af9d9b
Edit: There is even a config value
Take a histogram filter of a numeric column. Instead of fiddling around to drag the left and right handlers to it's position, you can just click on the reset button. Same goes for data mapping editor. |
Re 1 - yes. Re 2 - ok. Not sure how much better this is than just cancelling the whole operation and re-opening the dialog, especially given that it's not an "undo" as the button suggests but a "reset". Maybe in an ideal world, we would be able to do proper undo? And if we want to keep it, we should distinguish between simple dialogs that don't need that (like view specification) and more complex ones where it might make sense. |
When I choose a different visualization for a column, I see a preview of that immediately in lineup.
When I then click on the lineup view to make the menu disappear, the column reverts to the old visualization.
To confirm a visualization, I have to click a small check mark at the bottom:
Expected behavior
When I click away, the state of the preview is retain. That means, "apply" is the default action, not "cancel".
I don't think we need an undo button here. The undo button also doesn't do an expected "undo", but resets to the initial state. I can just as well cancel.
This should use proper buttons with suggested action highlight.
The text was updated successfully, but these errors were encountered: