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

Feature/upgrade to sb6 #1926

Merged
merged 35 commits into from
Aug 28, 2020
Merged

Feature/upgrade to sb6 #1926

merged 35 commits into from
Aug 28, 2020

Conversation

danyball
Copy link
Contributor

@danyball danyball commented Aug 19, 2020

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:

  • 'storybook-readme/register': 3rd party addon. Not working correctly with SB6. Nobody maintains this Addon.
    • removed that completly

wait to be fixed by others (#1935):

Done is when (DoD):

  • Modifications within components are reflected by tests.
  • A self-review of the code has been performed.
  • The modified component properties have been added to the typescript typings.
  • Potential design changes have been reviewed by a designer.
  • The modifications are well documented (README.md, etc.) and showcased in the stories.
  • The modifications are documented in the code, particularly in hard-to-understand areas (focus on WHY).
  • The modifications are shortly added to CHANGELOG.md with the issue number.
  • The modifications still work in IE.
  • The modifications are according to our Best Practices

@danyball danyball self-assigned this Aug 19, 2020
package.json Outdated Show resolved Hide resolved
@danyball
Copy link
Contributor Author

  • Remove readme code from stories
  • fix codepreview code (disabling) at stories

@danyball danyball force-pushed the feature/upgrade_to_sb6 branch from 4b90161 to 95051b6 Compare August 26, 2020 13:13
@danyball
Copy link
Contributor Author

TODO: make IE11 work great again:)

@danyball danyball added the R4R Ready for Review - Please review this PR. label Aug 27, 2020
@danyball danyball mentioned this pull request Aug 27, 2020
9 tasks
@@ -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
Copy link
Contributor

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.

Copy link
Contributor Author

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.

@danyball danyball merged commit 711b66d into develop Aug 28, 2020
@danyball danyball deleted the feature/upgrade_to_sb6 branch August 28, 2020 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R4R Ready for Review - Please review this PR.
Projects
None yet
2 participants