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

[feat] add deck.gl onFilteredItemsChange callback to DeckGl overlays #2691

Merged
merged 2 commits into from
Oct 15, 2024

Conversation

igorDykhta
Copy link
Collaborator

@igorDykhta igorDykhta commented Oct 12, 2024

  • add layerFilteredItemsChange to visState
  • add onFilteredItemsChange callback to deck.gl layers
  • save filteredItemCount to layer
  • add getFilterItemCount to layer
  • getSkipIndexes fix, wait for Time Sync

Copy link

netlify bot commented Oct 12, 2024

Deploy Preview for keplergl2 ready!

Name Link
🔨 Latest commit bbc94da
🔍 Latest deploy log https://app.netlify.com/sites/keplergl2/deploys/670ec0cfeab7750008845a4b
😎 Deploy Preview https://deploy-preview-2691--keplergl2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

igorDykhta and others added 2 commits October 15, 2024 22:20
Signed-off-by: Ihor Dykhta <dikhta.igor@gmail.com>
Signed-off-by: Ihor Dykhta <dikhta.igor@gmail.com>
@igorDykhta igorDykhta force-pushed the igr/ups-on-filtered-items-change branch from 56bace0 to bbc94da Compare October 15, 2024 19:21
@igorDykhta igorDykhta merged commit 1c0ef9a into master Oct 15, 2024
12 checks passed
@igorDykhta igorDykhta deleted the igr/ups-on-filtered-items-change branch October 15, 2024 21:25
MilheiroSantos pushed a commit to MilheiroSantos/kepler.gl that referenced this pull request Nov 1, 2024
…eplergl#2691)

* add layerFilteredItemsChange to visState
* add onFilteredItemsChange callback to deck.gl layers
* save filteredItemCount to layer
* add getFilterItemCount to layer

Signed-off-by: Ihor Dykhta <dikhta.igor@gmail.com>
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