Skip to content

Selection API and bug fix.#162

Merged
themustafaomar merged 4 commits intothemustafaomar:mainfrom
eswwaa:NewAPIs
Aug 9, 2024
Merged

Selection API and bug fix.#162
themustafaomar merged 4 commits intothemustafaomar:mainfrom
eswwaa:NewAPIs

Conversation

@eswwaa
Copy link
Copy Markdown
Contributor

@eswwaa eswwaa commented Aug 3, 2024

  1. Fix single selection problem where previously selected markers aren't deselected.

  2. Fix problem where selected markers aren't retrieved correctly during marker selection.

  3. Add a selectMarkers api method.

…n't deselected.

2. Add a selectMarkes api method.
@vercel
Copy link
Copy Markdown

vercel bot commented Aug 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
jvm-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 9, 2024 2:33pm

Clearing and select regions/markers explicitly as regions is not a private property, ther old way will error out becuase getSelected/clearSelected takes a _ as a prefix
@themustafaomar
Copy link
Copy Markdown
Owner

Hi @eswwaa

Thanks for brining this into my mind!

I've applied some changes:

  • Replaced selectMarkers with setSelectedMarkers for convetion.
  • Explicitly clear and select regions/markers, becuase regions is not a private property, ther older way will error out becuase getSelected/clearSelected takes a _ as a prefix.

@themustafaomar themustafaomar merged commit fa3208b into themustafaomar:main Aug 9, 2024
@eswwaa
Copy link
Copy Markdown
Contributor Author

eswwaa commented Aug 9, 2024

Thanks for getting this in themustafaomar, jsvectormaps is great. Any idea when a new package will be on npm?

@themustafaomar
Copy link
Copy Markdown
Owner

Maybe by the end of this month.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants