-
Notifications
You must be signed in to change notification settings - Fork 14
Description
Describe the feature
For customised maps (e.g. EPA), there's use cases for being able to override the default event handlers onMapSingleClick & onMapMove (e.g. to customise map centering, popup toggling or cluster handling). Currently activatePin is exposed but not overridable.
Motivation
EPA wished to disable map centering on pin click as the animation/move was visually distracting.
Proposal
Add the event handlers as optional properties on the component, with defaults set to the current handlers used.
Benefits
Easier implementation of custom functionality for sites requiring advanced map components, e.g. EPA.
Drawbacks
May need DGS review of any custom implementations in order to ensure conformance with any existing style/a11y guidelines or UX patterns.
Adoption strategy
No breaking changes, existing behaviour would be preserved via prop defaults.
Additional information
- Would you be willing to help implement this feature?
- Could this feature be created outside of Ripple-framework as a layer?
Final checks
- I have read the contribution guide for information on submitting a feature request.
- I have checked the discussions and issues for duplicate feature requests.