GabeLoins opened a new pull request #4610: [Explore] removing refresh chart 
overlay on deck_gl charts
URL: https://github.com/apache/incubator-superset/pull/4610
 
 
   Deck gl charts have this weird thing where scrolling the canvas isn't 
considered `renderTrigger: true` but also immediately updates the state of the 
chart. This scrolled state also does not persist across refreshes.
   
   I'm not sure what the long term solution here is, but for the short term I'm 
disabling the refresh chart overlay for deckgl graphs.
   
   test plan:
   - scroll on a deck gl chart
   - verify the state is updated and a refresh chart overlay does not appear
   
   reviewers:
   @john-bodley @michellethomas @mistercrunch 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to