-
Notifications
You must be signed in to change notification settings - Fork 588
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
Minor UI/UX Improvements #5719
Minor UI/UX Improvements #5719
Conversation
const NoLabelStyle = { | ||
display: 'inline', | ||
fontWeight: 'normal', | ||
color: '#6a6a6a' |
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.
Why not implement this in the stylesheet?
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.
Because these styles are used for a very specific use case. Here, its used to override default styles inherited from the form > label CSS. If you want I can add it as a CSS class but it will be hardly used again.
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.
I would prefer this to be done in CSS here.
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.
Sure
Can you include a before/after comparison? |
Its a very minor thing but adds to the overall user experience. With these changes, the user can select the radio button using the label. In this video notice first the production behavior and then the updated changes. screen-recording-2024-03-26-22.29.webm |
What this PR does
I have created this PR to fix some minor UI/UX bugs that I might come across while working on other tasks.
Changes
You can Merge this PR now or wait till the PR has some more significant commits.