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
Give our RadioButton a retouch #3791
Give our RadioButton a retouch #3791
Conversation
It now shares the same transition time and "bounce" as the CheckBox, which makes them resemble each other far more. Its label now aligns itself better as well.
No more custom labels.
ABA-135 Improve RadioButton component
Since the Checkbox component now looks flawless, the radio buttons look awful … |
@@ -1,90 +1,95 @@ | |||
.box { |
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.
The entire file is rewritten, so there's no need to compare changes.
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.
Looks much more like the checkboxes, I like it :))
Cleaner code also in Joblisting, which is always nice
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.
Awesome!
label="Frist" | ||
checked={order.deadline} | ||
onChange={() => { | ||
props.history.push({ |
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.
Would be nice if you could replace these with replace
as well 😄
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.
Out of scope here. Write it up as a todo and we can go over all of these
Description
It now shares the same transition time and "bounce" as the
CheckBox, which makes them resemble each other far more.
Its label now aligns itself better as well.
Result
Before
Screen.Recording.2023-04-15.at.00.14.08.mov
After
Screen.Recording.2023-04-15.at.00.10.55.mov
Screen.Recording.2023-04-15.at.00.11.38.mov
Testing
Tested radio buttons in and outside of forms.