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

Add options to discover initial color mode from URL param #79

Merged
merged 2 commits into from
Feb 16, 2024

Conversation

mondeja
Copy link
Contributor

@mondeja mondeja commented Feb 11, 2024

Closes #78

I'm not sure if this is the best way to implement it, but seems to work fine. Suggestions would be appreciated.

@mondeja mondeja changed the title Add initial_value_from_url_param option to UseColorModeOptions Add initial_value_from_url_param to UseColorModeOptions Feb 11, 2024
src/use_color_mode.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@maccesch maccesch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good. I have added a few minor remarks

src/use_color_mode.rs Outdated Show resolved Hide resolved
src/use_color_mode.rs Outdated Show resolved Hide resolved
@mondeja mondeja changed the title Add initial_value_from_url_param to UseColorModeOptions Add options to discover color mode from URL param Feb 15, 2024
@mondeja mondeja force-pushed the initial_value_from_url_param branch 2 times, most recently from f68bfb5 to b8b3e78 Compare February 15, 2024 21:44
@mondeja mondeja force-pushed the initial_value_from_url_param branch from b8b3e78 to 3109a54 Compare February 15, 2024 21:45
@mondeja
Copy link
Contributor Author

mondeja commented Feb 15, 2024

Now looks much better.

@mondeja mondeja changed the title Add options to discover color mode from URL param Add options to discover initial color mode from URL param Feb 15, 2024
@maccesch maccesch merged commit 3ffe3da into Synphonyte:main Feb 16, 2024
2 checks passed
@maccesch
Copy link
Contributor

Nice work, thanks!

@mondeja mondeja deleted the initial_value_from_url_param branch February 16, 2024 02:57
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.

Discover initial color mode from URL parameter
2 participants