Skip to content
This repository has been archived by the owner on Dec 13, 2017. It is now read-only.

fix: remove background fix position #16

Closed
wants to merge 1 commit into from
Closed

fix: remove background fix position #16

wants to merge 1 commit into from

Conversation

vidaaudrey
Copy link

Wrapper with fixed position will prevent the component area of storybook from scrolling down.

@johnthepink
Copy link

@vidaaudrey thanks for the PR! We will get this reviewed soon.

Copy link
Contributor

@jbaxleyiii jbaxleyiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vidaaudrey if we remove position: fixed then the background doesn't take up the entire container. We should improve support for scrolling contents however.

Would you be able to adjust this PR to allow the background to still cover the entire preview and allow for scrolling?

I'd be happy to help if you need any guidance!

@vidaaudrey
Copy link
Author

Hi @jbaxleyiii sorry for my late reply. I'm not sure what's the best way to fix the problem. Could you let me know? Thanks.

@jbaxleyiii
Copy link
Contributor

@vidaaudrey I think #17 will actually fix the issue! I'm going to release it today so we can see!

@jbaxleyiii jbaxleyiii closed this Oct 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants