Skip to content

Updated SilkyRGB API - #4

Open
chrizzo-hb wants to merge 34 commits into
doughn0:mainfrom
chrizzo-hb:main
Open

Updated SilkyRGB API#4
chrizzo-hb wants to merge 34 commits into
doughn0:mainfrom
chrizzo-hb:main

Conversation

@chrizzo-hb

Copy link
Copy Markdown
Contributor

I made some changes to the API, I hope you don't mind:

  • I exposed the list of named colors through a new get-colors route
  • I extended the payload of the get-palettes route to contain the color names
  • I renamed some settings and replaced palette.invert with color.primary/color.secondary
  • Additionally to setting a palette, users can now also set a primary/secondary color

In the ES menu, we now have 3 option lists:

  • Palette lets you pick a palette
  • Primary Color is automatically set to primary color of selected palette when a palette is selected
  • Secondary Color is automatically set to secondary color of selected palette when a palette is selected
  • Primary Color and Secondary Color can be changed by the users - in this case, Palette is automatically changed to Custom
  • For SilkyRGB, only led.color.primary and led.color.secondary should be relevant now (it is still possible to also set a palette directly though, I did not change that, I only override it with primary/secondary color for now)

This gives users more flexibility to create their own custom palette, yet they can still benefit from the awesome presets you created!!

However, since they can now manually switch primary and secondary color anyway, the invert option is obsolete so I removed it.

I believe this change will streamline the API and make it a bit less confusing, yet more flexible for users.

I hope you agree with this change?

chrizzo-hb and others added 30 commits March 2, 2026 00:36
Small optimizations for joystick mode. Biggest gain is when joysticks aren't being used, cpu usage is much lower.
fix initial led flash/blink before effect takes over
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.

2 participants