MudPicker: Set TextUpdateSuppression based on Readonly state#11144
MudPicker: Set TextUpdateSuppression based on Readonly state#11144ScarletKuro merged 3 commits intoMudBlazor:devfrom
Conversation
|
@henon @ScarletKuro MudBlazor/src/MudBlazor/Base/MudBaseInput.cs Lines 494 to 504 in df259cb any reason _isFocused should not be set to false when OnBlurredAsync is called and ReadOnly is true? I'd expect line 504 to be moved above the ReadOnly condition
|
Yes, good catch! |
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #11144 +/- ##
=======================================
Coverage 91.09% 91.10%
=======================================
Files 436 436
Lines 14096 14096
Branches 2726 2726
=======================================
+ Hits 12841 12842 +1
Misses 639 639
+ Partials 616 615 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|



Description
TextUpdateSuppression defaults to
trueinMudBaseInput. Based on #1012, this was added to prevent issues with data input in BSS. When the Picker is set toReadOnlyhowever, users aren't able to directly input data into the input and text suppression is not needed. This change disables TextUpdateSuppression based on theReadOnlystate of theMudPickerFixes #6779
Fixes #6968
Fixes #9090
How Has This Been Tested?
Tested visually, occurs only on BSS
Before
before.fix.mp4
After
after.fix.mp4
Type of Changes
Checklist
dev).