Skip to content

Conversation

@livingkurt
Copy link
Collaborator

No description provided.

@Unreal-Dan
Copy link
Collaborator

definitely a lot less than I was expecting you to change, is that it?

@livingkurt
Copy link
Collaborator Author

definitely a lot less than I was expecting you to change, is that it?

No not yet, I need to strip down the color selection menu to just be a classic 20 color selection menu, with no edit or delete or HSV
My programmer stopped working for some reason so Im having to rewire up my arduino uno, which I havent had time yet to do

- Add new color constants: RGB_AMBER and RGB_CHARTREUSE
- Modify default color sets and pattern configurations
- Update Makefile for macOS programmer settings
…I need to make it exit early while clearing the previous color
- Add `current_color_set` to track selected colors during color selection
- Modify color selection logic to use `current_color_set` for storing and saving colors
- Ensure all selected colors are saved when exiting color selection mode
- Move dimmed selection indicator to end of method
- Simplify color selection state transition logic
- Ensure consistent visual feedback during color selection
- Implement `setAll` method in Colorset to replace manual color setting loops
- Modify color selection state to use new `setAll` method
- Simplify color setting logic in Helios state handling
- Improve color configuration efficiency and readability
- Modify `set` method to automatically add colors when setting an index beyond current palette size
- Simplify color setting logic by removing redundant index tracking
- Improve color palette management with more flexible color addition
- Remove unused pattern IDs from the enumeration
- Consolidate strobe-related patterns
- Reduce complexity of pattern identification
@Unreal-Dan Unreal-Dan marked this pull request as draft March 7, 2025 21:51
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 this pull request may close these issues.

3 participants