-
Notifications
You must be signed in to change notification settings - Fork 17
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
Feature/upgrade to sb6 #1926
Feature/upgrade to sb6 #1926
Conversation
|
…r that" This reverts commit fd0f582
4b90161
to
95051b6
Compare
TODO: make IE11 work great again:) |
@@ -67,7 +67,7 @@ test('should responsive stretch', async t => { | |||
]; | |||
}); | |||
|
|||
const expected = JSON.stringify(['529px', '0px']); // 529px width of the radio correspond to 100% in a 575px window | |||
const expected = JSON.stringify(['545px', '0px']); // 545px width of the radio correspond to 100% in a 575px window |
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.
The last number was forgotten, maybe just replace this comment with 100% width
or something.
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.
You mean 575px? Thats correct. At this test the browser is resized to 575px width.
Fixes #1774
fixes #1877
fixes #1900
Info: Many files have changed because i switched the "old" story.js format to the new one "CSF". You do not have to review ALL the story/demo files. But please have a look at all other changes.
unable to fix:
wait to be fixed by others (#1935):
HTML
Tab label: 2 Feature Requests whitespace-se/storybook-addon-html#27src
folder to our project. But than it is a "custom addon" and this causes another error (details below).Done is when (DoD):