-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
MudTooltip: Add Disabled parameter #8876
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #8876 +/- ##
==========================================
+ Coverage 89.82% 90.21% +0.38%
==========================================
Files 412 424 +12
Lines 11878 12286 +408
Branches 2364 2406 +42
==========================================
+ Hits 10670 11084 +414
+ Misses 681 669 -12
- Partials 527 533 +6 ☔ View full report in Codecov by Sentry. |
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.
Good idea, I wonder why it was never added before.
I think the test should check onmouseenter
too and explicitly enable ShowOnHover and ShowOnFocus for the corresponding tests to ensure it's capturing the right context. The Visible property should never be set to true while disabled and that should be tested as well.
@danielchalmers Do you mean I need to modify/add the logic in the |
I was thinking that the I'm not sure what should happen if the user uses the |
All 5
I believe that nothing should happen when |
My point is from a public API perspective where someone might rely on the |
Do you think you could take a look at #4466 as well? Seems like a simple fix. |
By default |
In all other components |
Thanks Yomodo! FYI @ScarletKuro |
Description
Resolves #8842
How Has This Been Tested?
Added 3 unit tests
Added 1 visual test
Type of Changes
Checklist
dev
).