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

feat: Add ability to toggle multiple light/dark classes #220

Merged

Conversation

chris-dura
Copy link
Contributor

@chris-dura chris-dura commented Feb 17, 2023

BREAKING CHANGE: lightClass and darkClass have changed from string to string[]. Before: darkClass: 'my-dark-class', after: darkClass: ['my-dark-class']

@chris-dura chris-dura force-pushed the feature/multiple-classes branch from 79ccbd8 to 5c04891 Compare February 17, 2023 02:35
BREAKING CHANGE: `lightClass` and `darkClass` have changed from `string` to `string[]`. Before: `darkClass: 'my-dark-class'`, after: `darkClass: ['my-dark-class']`
@chris-dura chris-dura force-pushed the feature/multiple-classes branch from 5c04891 to c5d1b17 Compare February 17, 2023 02:36
@hipstersmoothie
Copy link
Owner

Could you arrayify the class instead?

@chris-dura
Copy link
Contributor Author

Could you arrayify the class instead?

yep, I'm just not a fan of union types, so wanted to get your opinion :)

@chris-dura chris-dura changed the title feat!: Toggle multiple light/dark classes feat: Add ability to toggle multiple light/dark classes Feb 17, 2023
@chris-dura chris-dura marked this pull request as ready for review February 17, 2023 22:57
@chris-dura chris-dura marked this pull request as draft February 17, 2023 22:59
@chris-dura chris-dura marked this pull request as ready for review February 17, 2023 23:22
@chris-dura chris-dura marked this pull request as draft February 17, 2023 23:23
@chris-dura
Copy link
Contributor Author

This should be a backwards compatible way to support both a single 'myClass' string, a ['myClass'] array, and a multiple string ['my', 'class', 'names'] array.

@chris-dura chris-dura marked this pull request as ready for review February 17, 2023 23:35
@hipstersmoothie hipstersmoothie added the minor Increment the minor version when merged label Feb 19, 2023
@hipstersmoothie
Copy link
Owner

@chris-dura Thanks for the contribution!

@hipstersmoothie hipstersmoothie merged commit 1209f25 into hipstersmoothie:master Feb 19, 2023
@hipstersmoothie
Copy link
Owner

🚀 PR was released in v2.1.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Increment the minor version when merged released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants