Skip to content
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

feat(reset): set preview margins and paddings to 0 by default #19

Merged
merged 4 commits into from
Dec 16, 2020

Conversation

rbardini
Copy link
Owner

Storybook 6.1 introduces the layout parameter, set to padded by default.

This PR sets the preview margins and paddings to 0 by default, so users are not required to change the layout option to fullscreen when using the addon. It also ensures that both are reset in older Storybook versions, or when the none option is used.

@rbardini rbardini added the enhancement New feature or request label Dec 13, 2020
@rbardini rbardini self-assigned this Dec 13, 2020
@rbardini rbardini changed the title feat: reset preview margins and paddings by default feat(reset): set preview margins and paddings to 0 by default Dec 13, 2020
@joanrm20
Copy link
Contributor

hey @rbardini, are you planning to merge this first or #10 ?

@rbardini
Copy link
Owner Author

I think it would be better to merge this one first, so users don't need to upgrade to a major version to have it.

It might conflict with #10 though—I decided to use ranges in all example dependencies, so we always deploy with the latest supported versions. Is it ok for you?

@joanrm20
Copy link
Contributor

hey @rbardini, sounds good. After you merge this I can fix conflicts, we can test and merge #10.

@rbardini rbardini merged commit f31361f into master Dec 16, 2020
@rbardini rbardini deleted the reset-spacings branch December 16, 2020 16:54
@github-actions
Copy link

🎉 This PR is included in version 3.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants