-
Notifications
You must be signed in to change notification settings - Fork 27
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
Test change proposal for pointer events #628
Comments
Per @smaug---- Generally we think these tests will likely require some work to correctly match the latest specs, but that adding them to the focus area makes sense since this is an area where implementation differences are regularly causing real world difficulties for users and web authors. |
The spreadsheet with comments/motivations is at https://docs.google.com/spreadsheets/d/1GVqvTfRBh_film7kzMflCSJjIkq1TV4mRcPoldNSK1Q/edit?usp=sharing The tests that have "recent consensus" implies that the spec hasn't been fixed yet to reflect the consensus. Where that is the case, I think it would be good to get the spec updated before including the corresponding test in Interop. |
I will update the sheet now based on the comments above. If anyone wants to update the sheet, feel free to add your comment in Column F along with your email.
Thanks for calling this out, we have two such tests. After revisiting the spec status for both just now, I concluded that they are safe to "include". Please let us know if anyone thinks otherwise: |
Done. |
Thanks. OK for us to include. |
@nt1m / @aprotyas could you review the updated scope - pointerevents and uievents |
@nairnandu The 2024 dashboard entry for pointer/mouse events still point to the 2023 list. Is it time to include the added scope from your last comment? |
@nairnandu apologies for the delay here -- both scope changes look reasonable to me. I think we can proceed to reflecting this in the 2024 dashboard. |
Thank you @aprotyas and @mustaqahmed |
Test List
Rationale
As part of the carryover evaluation - #472 a new set of tests were proposed to be added. This was not reviewed during the carryover discussion. Raising it as a test change proposal.
The text was updated successfully, but these errors were encountered: