reynoldmorel commented on code in PR #37501:
URL: https://github.com/apache/superset/pull/37501#discussion_r2742000564
##########
superset-frontend/src/explore/components/ExploreChartPanel/index.tsx:
##########
@@ -99,29 +99,50 @@ const MIN_SIZES: PanelSizes = [300, 65];
const DEFAULT_SOUTH_PANE_HEIGHT_PERCENT = 40;
const Styles = styled.div<{ showSplite: boolean }>`
+ box-sizing: border-box;
display: flex;
flex-direction: column;
align-items: stretch;
align-content: stretch;
- overflow: auto;
Review Comment:
I think setting overflow to hidden is not a good option, because I can
change either size of the browser and the zoom, and those should make the
scrollbars to show, if there is a very small space to render the element. If
not it would look like cut
--
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]
