-
Notifications
You must be signed in to change notification settings - Fork 799
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
Map Block: Tweak the scroll-to-zoom behaviour. #14673
Conversation
Caution: This PR has changes that must be merged to WordPress.com |
Thank you for the great PR description! When this PR is ready for review, please apply the Scheduled Jetpack release: February 17, 2020. |
pento, Your synced wpcom patch D38908-code has been updated. |
pento, Your synced wpcom patch D38908-code has been updated. |
Thanks for the review, @glendaviesnz! f6143f5 fixes that up, looks like I needed to save the attribute onto the |
A note on there only being one item in the Settings panel, which looks a bit lonely: #14557 is also on my radar, that would go in it, too. |
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.
I've left a nitpick comment, but otherwise this LGTM!
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.
This works well in my tests! 👍
pento, Your synced wpcom patch D38908-code has been updated. |
Ugh, auto-dismissing reviews is the worst, especially for a minor fix. @jeherve, could you approve it again, so I can merge? |
pento, Your synced wpcom patch D38908-code has been updated. |
* 8.3 release: changelog * Changelog: add #14516 * Changelog: add #14574 * Bring in changes from 8.2.1 and 8.2.2 * Update stable version * Bring in 8.2.3 changes * Changelog: add #14714 * Changelog: add #14639 * Changelog: add #14678 * Changelog: add #14673 * Changelog: add #14687 * Changelog: add #14704 * Changelog: add #14702 * Changelog: add #14541 * Changelog: add #14657 * Changelog: add #14622 * Changelog: add #14582 * Changelog: add #14638 * Changelog: add #14633 * Changelog: add #14571 * Changelog: add #14592 * Changelog: add #14539 * Changelog: add #14514 * Changelog: add #14643 * Changelog: add #14494 * Changelog: add #13739 * Changelog: add #14707 * Changelog: add #14736 * Changelog: add #14706 * Changelog: add #14730 * Changelog: add #14685 * Changelog: add #14727 * Changelog: add #14711 * Changelog: add #14742 * Changelog: add #14746 * Changelog: add #14725 * Changelog: add #13999 * Changelog: add #14740 * Changelog: add #14759 * Changelog: add #14703 * Changelog: add #14753 * Changelog: add #14754 * Changelog: add #14645 * Cahngelog: add #14599
This PR changes the behaviour of scroll-to-zoom on the map block.
Currently, it will capture the scroll event whenever the pointer is over the map block: this means that you can accidentally zoom when scrolling through a post.
In the block editor, it now only zooms when the block is selected: scrolling through a post won't accidentally be captured by an unselected map block.
On the front end, the there is now an option (defaulting to off) for enabling scroll to zoom.
Fixes #11904.
Changes proposed in this Pull Request:
Is this a new feature or does it add/remove features to an existing part of Jetpack?
Testing instructions:
Screenshots
Proposed changelog entry for your changes: