Skip to content
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

Stream Deck Plus support + some fixes #16

Merged
merged 3 commits into from
Jan 28, 2024
Merged

Conversation

K900
Copy link
Contributor

@K900 K900 commented Nov 21, 2023

Add new message types for all the new Stream Deck+ stuff, fix color parsing for the latest versions of the app, update docs links

@kristofferpeterhansel
Copy link

Hehe. Very similar to the changes in my fork.

The color issue is that there is now an alpha channel in the color. So a fourth number (which can also be seen in the sample on the site: https://docs.elgato.com/sdk/plugins/registration-procedure#info-parameter).

My hack so far is just to accept the extra two bytes but ignoring them. The documentation does not seem very clear if it will always be there or not.

@K900
Copy link
Contributor Author

K900 commented Jan 11, 2024

@mdonoughe can you take a look?

src/registration.rs Outdated Show resolved Hide resolved
@K900 K900 force-pushed the main branch 2 times, most recently from 83c5ade to 9883d0d Compare January 14, 2024 06:13
@K900 K900 changed the title [wip] More Stream Deck Plus support More Stream Deck Plus support Jan 14, 2024
@K900
Copy link
Contributor Author

K900 commented Jan 14, 2024

Added proper RGBA colors support, also added docs for the new SD+ stuff.

@K900 K900 changed the title More Stream Deck Plus support Stream Deck Plus support + some fixes Jan 14, 2024
@K900
Copy link
Contributor Author

K900 commented Jan 14, 2024

I finally got around to fixing the very obvious issues in my plugin that uses this, so https://gitlab.com/K900/streamdeck-now-playing is now up if anyone is interested in a code sample.

@K900
Copy link
Contributor Author

K900 commented Jan 28, 2024

Re-ran rustfmt, no other changes.

@mdonoughe mdonoughe merged commit 9b8be4e into mdonoughe:main Jan 28, 2024
2 checks passed
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