-
Notifications
You must be signed in to change notification settings - Fork 2
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
Reset Diff Toggles for New Stories #187
Reset Diff Toggles for New Stories #187
Conversation
6bf4eed
to
ea86b92
Compare
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.
Code changes LGTM but can we have some stories for these situations (new test, new mode, new browser?)
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.
Code changes LGTM but can we have some stories for these situations (new test, new mode, new browser?)
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.
Code looks good and QA worked as expected
ea86b92
to
ef8d100
Compare
6a53faf
to
b9d9130
Compare
b990c64
to
a1de870
Compare
Previously, if you changed to a story where the comparison is new and the Baseline toggle shows the baseline snapshot, there would be no image shown in the comparison viewer. This PR adjusts the behavior so that for new stories, the user always sees the latest snapshot.
To QA:
📦 Published PR as canary version:
1.1.10--canary.187.a1de870.0
✨ Test out this PR locally via:
npm install @chromatic-com/storybook@1.1.10--canary.187.a1de870.0 # or yarn add @chromatic-com/storybook@1.1.10--canary.187.a1de870.0