-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat: add file and color controller setting types #13669
base: main
Are you sure you want to change the base?
feat: add file and color controller setting types #13669
Conversation
These setting types are great features! For the file selector I would expect a widget like this: More complicated might be a generic definiton for colors, as:
|
Thanks for the early feedback!
Yes, that makes sense. It's in the todo list already :)
The current widget are more tailored for palette AFAIK, not unique colour selection, so not sure how to do this? I was thinking of adding a color indication inside/next to the pick button
Yes, this usecase is already covered with enum, and actually used by the S4Mk3. One bits of improvement could be to add some color indication in the dropdown, I guess we could easily add an attribute on the |
I think using the Qt color picker is fine. If we want to we can pre-fill their color with for example the users currently track-color palette (the palette source could also be configured as part of the setting). But lets KISS for now until we have a good reason to couple this to our palettes. A palette selector could be interesting indeed, but that is separate from the color picker. |
b167d82
to
74eebc4
Compare
a1cf79d
to
d91587e
Compare
Add new controller setting type, particularly useful for screen mapping.
Screencast.from.19-09-24.10.02.23.mp4
TODO: