-
Notifications
You must be signed in to change notification settings - Fork 612
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
feat: swap 'mouse' events to 'pointer' events for mobile compatibility; update site and docs #9089
Conversation
alliefeldman
commented
Sep 15, 2023
- Site examples now use ‘pointer’ events instead of mouse events; all other elements of examples remain unchanged
- Update ‘site/docs/’ to include ‘pointer’ events
- All site examples use ‘pointer’ instead of ‘mouse’ in the instances of -over, -move, -out, -down, -up
- Unit tests include pointer events
- All lint and unit tests pass
Can you add feat: to the title? |
Done! |
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.
Please clean up the pull request. Otherwise looks good.
…ction, and removed accidental site/ files
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.
Please look over the pull request more carefully. There are a lot of cahnges that should not be included.
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.
Please look over the pull request more carefully. There are a lot of cahnges that should not be included.
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.
Delete the versions file as well.
"size": { | ||
"condition": {"test": {"param": "hover", "empty": false}, "value": 48}, | ||
"value": 100 | ||
"mark": "line" |
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.
Let's leave the indentation as it was before.
…y; update site and docs (vega#9089)
…y; update site and docs (vega#9089)