mihir174 edited a comment on issue #11916: URL: https://github.com/apache/incubator-superset/issues/11916#issuecomment-738289897
@etr2460 Thanks for the quick feedback! I totally agree that the chart should have enough space. At these screen dimensions, the current panel has a width of 494px, and the proposed one is 576px (82px difference is about the length of the current `run` button for ref). We still haven't finalized the area split between the chart and the data, so there is an opportunity to increase the chart size there - what do you think is a good min. size for the chart or ratio between chart and data table areas? Here's the main layout issue around a collapsable panel illustrated: SIP-34 Design- Expanded  Collapsed - the panel just slides to the left  This design (if we were to have the section collapse)- Expanded  Collapsed - the panel has to move under the datasource selector, which is a really awkward motion  ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
