Refactor ColorIndicator #159
Conversation
- Fixed several bugs -- enabled is now taken as a ref -- color is always popped (since deleted) - Added a custom rendering instead of abusing the Checkbox and changing some colors -- Some cases still not handled such as disabled (not interactive) and hover while it is active
|
@matt-edmondson The colour modification options in ImGuiStyler are quite extensive but I'm not sure what I should use when I want to take a defined colour and turn it into a hovered version. I tried LightenBy but that didn't really look any good. Any recommendations or guidance? |
The intention of this widget is to be non-interactive by default. Remove the click/hover logic and just display a color or framebg depending on if it is enabled
We talked about this on the phone and decided that |
|
And then we decided we didn't actually need any of that because the widget is intended to be non-interactive anyway. |
-- color is always popped (since deleted)