-
-
Notifications
You must be signed in to change notification settings - Fork 998
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(module select): simulate setTimeout, ensure that the click event is processed properly #3525
fix(module select): simulate setTimeout, ensure that the click event is processed properly #3525
Conversation
2111124
to
443f6ea
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #3525 +/- ##
==========================================
- Coverage 44.47% 44.08% -0.40%
==========================================
Files 566 566
Lines 27182 27218 +36
Branches 5577 5583 +6
==========================================
- Hits 12090 11999 -91
- Misses 14101 14233 +132
+ Partials 991 986 -5 ☔ View full report in Codecov by Sentry. |
{ | ||
await Task.Delay(4); |
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 is it 4ms?
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.
In JS, 4ms is a better configuration, and I think the same goes for Blazor.
https://html.spec.whatwg.org/multipage/timers-and-user-prompts.html#dom-settimeout
8eed893
to
957cfe7
Compare
baba816
to
b3241cf
Compare
b3241cf
to
8785f15
Compare
This have been fixed via #3543 |
🤔 This is a ...
🔗 Related issue link
#3521
💡 Background and solution
📝 Changelog
☑️ Self Check before Merge