-
Notifications
You must be signed in to change notification settings - Fork 85
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: set initial value using property to ensure change is fired #6606
Conversation
Kudos, SonarCloud Quality Gate passed!
|
The solution looks good to me. Although, I wasn't able to reproduce the issue in my Safari (version 17.0 build 19616.1.27.211.1), which makes me think that it was a browser bug that got fixed. I've recently upgraded to Mac OS Sonoma. |
For the record, the test fails on
|
… (#6609) Co-authored-by: Serhii Kulykov <[email protected]>
… (#6608) Co-authored-by: Serhii Kulykov <[email protected]>
… (#6607) Co-authored-by: Serhii Kulykov <[email protected]>
Description
Fixes #6224
Fixes #6598
This PR fixes an issue in Safari with
change
event not getting fired for clearing initial value set with attribute.See #6224 (comment) for the HTML snippet that I used to reproduce and debug this issue.
Type of change