-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix initial bottom panel size #6822
Conversation
… dynamic to the screen size
This pull request is automatically deployed with Now. Latest deployment for this branch: https://monorepo-git-fork-mortal94-fix-initial-panel-size.storybook.now.sh |
@@ -282,7 +282,7 @@ class Layout extends Component { | |||
resizerPanel: | |||
resizerPanel || | |||
(options.panelPosition === 'bottom' | |||
? { x: 0, y: bounds.height - 400 } | |||
? { x: 0, y: bounds.height * 0.6 } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to round this to a whole number, to not get inconsistencies
Thanks @mortal94 this is great, one small remark, then good to merge IMHO |
Thanks @ndelangen ! i've pushed the requested changes. |
@mortal94 yes this will be released in beta.2 after it gets merged |
Issue:
The initial bottom panel size on small screens was too big
What I did
Changed the initial value of the bottom panel size to be dynamic to the screen.