fix(deck.gl): update view state on property changes (#17720)

[hbruch]

SUMMARY

Updates viewport state of deck.gl on filter updates

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before (screeshot 1 & 2):

After:

update_viewport_480.mov

TESTING INSTRUCTIONS

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

ADDITIONAL INFORMATION

• Has associated issue: fixes deck.gl polygon doesn't auto zoom to changed bounds when native filter is updated #17720
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #17826 (dbbfec7) into master (6991417) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #17826      +/-   ##
==========================================
- Coverage   67.07%   67.07%   -0.01%     
==========================================
  Files        1609     1609              
  Lines       64899    64901       +2     
  Branches     6866     6867       +1     
==========================================
  Hits        43533    43533              
- Misses      19500    19502       +2     
  Partials     1866     1866              

Flag	Coverage Δ
javascript	53.76% <0.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...legacy-preset-chart-deckgl/src/DeckGLContainer.jsx	0.00% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6991417...dbbfec7. Read the comment docs.

[villebro]

Thanks for the fix! Works well, but minor performance improvement suggestion

[villebro]

I believe this will cause unnecessary rerenders as the object reference may change despite its properties being unchanged. Let's use lodash here, something like

import { isEqual } from 'lodash';

if (!isEqual(nextProps.viewport, this.props.viewport)) {
  ...
},

[hbruch]

Thanks for the hint. Is adressed by last commit (though checks failed again)...

[villebro]

@hbruch can you rebase this? I believe there may have been some CI issues when you pushed these last changes

[villebro]

@hbruch sorry about this, but there's yet some more flakiness in our CI pipeline: #17918 . Bear with us while we get it sorted..

[hbruch]

@villebro Is there anything I could do to have this merged? Thx

[villebro]

Oh, sorry about the delay!