-
Notifications
You must be signed in to change notification settings - Fork 10
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
Improve toggle switch #22
Comments
Updates. |
What's the diffrence with the "Better formatting"? The helper text variant can be solved with the general Bootstrap layout utilities, we do not ned to define a custom component for that imho. |
Can we close this? As far as I see we don't need to customize the component and should use the Bootstrap utils for applying the helper text. |
That's a fair point .. took a look at the code of how we're implementing the switches and agree with you, as long as it's consistent in the implementation, which it seemed to be, I'm cool with that. I suppose the only other changes to the switches could be off/on hover states, but that's just more minor interactive stuff that could be done at any time. |
The text was updated successfully, but these errors were encountered: