Re: [PR] feat(dashboard): Hide chart header controls in reports [superset]

["john-bodley (via GitHub)" <gi...@apache.org>]

john-bodley commented on PR #26035:
URL: https://github.com/apache/superset/pull/26035#issuecomment-1819551397

   @kgabryje I think having the filter indicators et al. in the render report or alert is actually helpful to the reader as it indicates which charts are impacted by the filters. Without them the results could more easily* be misinterpreted.
   
   \* Note even currently these can still be misinterpreted given it's not overly evident (without hovering) which filters are impacting said chart.


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org