-
Notifications
You must be signed in to change notification settings - Fork 538
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 color picker option to accept changes on dismissal #1635
base: master
Are you sure you want to change the base?
Conversation
@@ -148,6 +148,13 @@ public enum PlayAction | |||
[EditorDisplay("Interface"), EditorOrder(280), Tooltip("Editor content window orientation.")] | |||
public FlaxEngine.GUI.Orientation ContentWindowOrientation { get; set; } = FlaxEngine.GUI.Orientation.Horizontal; | |||
|
|||
/// <summary> | |||
/// Gets or sets the editor content window orientation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably a copy and paste error here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed, will fix
/// </summary> | ||
[DefaultValue(false)] | ||
[EditorDisplay("Interface"), EditorOrder(300), Tooltip("If checked, color pickers will always modify the color unless 'Cancel' if pressed, otherwise color won't change unless 'Ok' is pressed (default)")] | ||
public bool ColorPickerAlwaysChangesColor { get; set; } = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about like AutoAcceptOnColorChange
or AutoPickOnColkrChange
or AutoAcceptColorPickerColor
or something? Idk naming things is hard.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, there is nothing really automatic about the option, is it? You still pick the color, is just that the change doesn't need "acknowledgement", but not sure.
Maybe we can say "Color picker magical option 42, surprise!" or something... 😁
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That said "Auto accept color picker change" might just work. What do you think?
I'm really used to the Unity (and maybe other creative tools) of color picker functionality, where you don't need to press "Ok" for the color to be accepted, just dismissing the dialog is enough.
This PR adds an option to be able to do that, and I'd be grateful if somebody has some feedback on the naming, category location and/or tooltip description, naming is difficult.