michael-s-molina commented on PR #19558: URL: https://github.com/apache/superset/pull/19558#issuecomment-1093021777
> When a control that's not a "render trigger" changes, user sees an alert banner. That lets them know that they should click update chart button, but at the same time they can still see their chart. That's better than the overlay! I like that the user can see the chart. I would just add the update button right there instead of floating on the left panel. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
