-
Notifications
You must be signed in to change notification settings - Fork 320
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
Fix inline radios #607
Fix inline radios #607
Conversation
CH CH CH CHANGELOG :) sorry :( |
Introduce a modifier at the block level which makes all radios within that block inline. You can’t add classes to the individual radio items using the macro, and it doesn’t really make sense to require the modifier to be applied to every individual element. Add an example of inline radios so that they are represented in the review app. Update the readme.
03baa44
to
db879d7
Compare
Updated. 🎵Just gonna have to be a different man |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🆕
Sorry @igloosi – one more commit to check. |
it's good, though i feel like using inline radios is when you have shortly worded selection which should fit side-by-side on mobile. |
Don't disagree, but from talking to @dashouse I think matching the existing behaviour in Elements is the right thing to do for now. |
Turns out we do actually have a style for inline radios, it's just not documented or easy to apply. (h/t to @dashouse for spotting this…)
Introduce a modifier at the block level which makes all radios within that block inline. You can’t add classes to the individual radio items using the macro, and it doesn’t really make sense to require the modifier to be applied to every individual element.
Add an example of inline radios so that they are represented in the review app.
Update the readme.