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

deck.gl polygon doesn't auto zoom to changed bounds when native filter is updated #17720

Closed
3 tasks done
hbruch opened this issue Dec 12, 2021 · 0 comments · Fixed by #17826
Closed
3 tasks done

deck.gl polygon doesn't auto zoom to changed bounds when native filter is updated #17720

hbruch opened this issue Dec 12, 2021 · 0 comments · Fixed by #17826
Labels
#bug Bug report validation:required A committer should validate the issue

Comments

@hbruch
Copy link
Contributor

hbruch commented Dec 12, 2021

A clear and concise description of what the bug is.

How to reproduce the bug

  1. Create a Dashboard containing a deck.gl polygon chart (auto zoom = True, screenshot 1) with a column included in filter
  2. Open the dahsboard => the map is zoomed according to show features (screenshot 2)
  3. Change native filter value
  4. The map is not zoomed to the new bounding box (screenshot 3)

Expected results

The map should be zoomed to the new bounding box

Actual results

The map should is not zoomed to the new bounding box

Screenshots

grafik

Screenshot 1

grafik

screenshot 2

grafik

screenshot 3

Environment

  • browser type and version: Firefox 94.0.1
  • superset version: master head

Checklist

Make sure to follow these steps before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • I have reproduced the issue with at least the latest released version of superset.
  • I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context

Add any other context about the problem here.

@hbruch hbruch added the #bug Bug report label Dec 12, 2021
@geido geido added the validation:required A committer should validate the issue label Jan 25, 2022
villebro pushed a commit that referenced this issue Feb 11, 2022
* Update viewstate on property changes

* Compare props via isEqual
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug Bug report validation:required A committer should validate the issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants