-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add tooltip to arbitrary lines UI #211
Conversation
…explanatory tooltip for value input of arbitrary lines ui
So I didn't realize until submitting this PR that teal has a class. I can update to use this class tomorrow. |
Code Coverage Summary
Diff against main
Results for commit: b416519 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
I like the work 👍 Some modules in |
Nice! I hadn't seen that. This makes me feel even more like we ought to consider a standard way to do this. Maybe a function like here or even just documenting somewhere to use CSS classes provided by |
…a CSS class provided by teal
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.
Yup 👍
@asbates can this be merged? |
with_tooltip
Fixes #111
According to the Bootstrap docs (BS5, BS3), tooltips are opt-in and must be initialized individually. However, I am seeing tooltips display even without explicitly initializing them. Anyone know why? Can we rely on this always being the case or should we run JS to initialize?