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

fix(map): cluster problem #1172

Open
wants to merge 1 commit into
base: fix/SVA-1399-fix-bugs-in-map-package
Choose a base branch
from

Conversation

ardasnturk
Copy link
Member

@ardasnturk ardasnturk commented Jul 30, 2024

  • added ShapeSource component instead of PointAnnotation component to solve clustering bug in android
  • Added clustersData in renderCluster to create the desired data format of the shape in ShapeSource
  • added value 1000 to the fitBounds function to show the cluster in an animated way
  • updated with parseInt inside handleClusterPress since cluster.id is string in android

SVA-1399

FIXME: point_count inside the cluster

before after
Screenshot_1722344429 Screenshot_1722344393

@ardasnturk ardasnturk added the bug Something isn't working label Jul 30, 2024
@ardasnturk ardasnturk added this to the 4.0.1 milestone Jul 30, 2024
@ardasnturk ardasnturk self-assigned this Jul 30, 2024
- added `ShapeSource` component instead of `PointAnnotation` component to solve clustering bug in android
- Added `clustersData` in `renderCluster` to create the desired data format of the `shape` in `ShapeSource`
- added value 1000 to the `fitBounds` function to show the cluster in an animated way
- updated with `parseInt` inside `handleClusterPress` since `cluster.id` is string in android

SVA-1399
@ardasnturk ardasnturk force-pushed the fix/SVA-1399-fix-cluster-problem branch from b1eaace to 56344e1 Compare July 30, 2024 13:42
type: 'FeatureCollection'
};

markers.map((item) => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you want to loop here, right, not map some value and return a new array? so please change to a loop over markers to update the values in clustersData.features

@donni106 donni106 added the invalid This doesn't seem right label Jul 31, 2024
@donni106
Copy link
Member

Stop this for now, clustering still not work properly, especially the display of the point counts.

@donni106 donni106 removed this from the 4.0.1 milestone Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants