Skip to content
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

Adding hover on pins #110

Merged
merged 28 commits into from Aug 11, 2022
Merged

Adding hover on pins #110

merged 28 commits into from Aug 11, 2022

Conversation

Feynallein
Copy link
Contributor

Hello,
I'm working on a web project that uses flutter_map_marker_cluster and adding the hovers on markers using clusters is an important feature of my project.
If buildPopupOnHover is set to true, a MouseRegion will encapsulates the marker.builder (within the GestureDetector) and the popup will be built when the marker is hovered. When tapping a Marker, only the onMarkerTap method will be called.
I also added two methods : onMarkerHoverEnter and onMarkerHoverExit which follow the spirit of onMarkerTap : they're called respectively on hover enter and on hover exit.
Finally, I added a timer before the popup is shown, this timer is set to 300 ms by default.

I hope we can do something for the hovering feature to be added as it can be very useful for web projetcs.

Thanks for reading!

@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Mar 13, 2022
@Feynallein
Copy link
Contributor Author

Up (to remove stale)

@github-actions github-actions bot removed the Stale label Mar 16, 2022
@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Apr 15, 2022
@Feynallein
Copy link
Contributor Author

Removing stale

@github-actions github-actions bot removed the Stale label Apr 22, 2022
@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label May 30, 2022
pubspec.yaml Outdated Show resolved Hide resolved
@lpongetti
Copy link
Owner

pls fix conflicts

@Feynallein
Copy link
Contributor Author

Conflicts has been resolved.
Branch has been updated.

@lpongetti lpongetti merged commit d7a296e into lpongetti:master Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants