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 and use the description
on fields
#4189
Fix and use the description
on fields
#4189
Conversation
ABA-494 Move tooltips on event editor to the side of the input field
Kind of annoying that all the fields here have a tooltip. A lot of the time it is not interesting since it has already been read. Instead, there should be an info icon or something (kind of like on the poll overview component) at the side. |
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.
Nice stuff! Some small knitpick then we good (basically only the comment about the deleted css selector - the others aren't necessarily deal breakers)
a2abc2a
to
f7946f4
Compare
f7946f4
to
591b559
Compare
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.
Lgtm
description
on fields
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.
LGTM
Description
Fix the description styling on fields
Refactor fields to use description prop
Result
The icons makes the page more "cluttered", but easier/better to use. It's a trade-off. Note that the event editor is the only page with a ton of field descriptions, so given that it's an anomaly I wouldn't say it's a big deal.
Testing
All fields still look as expected.