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

fix(toast): Remove transform origin for visual stories #611

Closed
wants to merge 2 commits into from
Closed

fix(toast): Remove transform origin for visual stories #611

wants to merge 2 commits into from

Conversation

mannycarrera4
Copy link
Contributor

@mannycarrera4 mannycarrera4 commented May 1, 2020

When Chromatic takes its snapshot, the Toast hasn't animated into view so the snapshots look empty, by allowing a transform origin of null, we get to have our snapshots back.

@mannycarrera4 mannycarrera4 changed the base branch from master to prerelease/v4 May 1, 2020 21:52
@cypress
Copy link

cypress bot commented May 1, 2020



Test summary

333 0 1 0


Run details

Project canvas-kit
Status Passed
Commit 914b7f9 ℹ️
Started May 4, 2020 6:36 PM
Ended May 4, 2020 6:40 PM
Duration 03:38 💡
OS Linux Ubuntu Linux - 18.04
Browser Electron 80

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@mannycarrera4 mannycarrera4 marked this pull request as ready for review May 4, 2020 15:15
@mannycarrera4 mannycarrera4 added the ready for review Code is ready for review label May 4, 2020
@NicholasBoll
Copy link
Member

This contains some of the same code as #600 which is against master. Should we just merge master forward and close this PR?

@mannycarrera4
Copy link
Contributor Author

Works for me @NicholasBoll

@mannycarrera4
Copy link
Contributor Author

Solved in #600 once we merged master forward into pre release v4

@anicholls anicholls added the blocked This issue is waiting for one or more issues to be closed/resolved. label May 5, 2020
@anicholls
Copy link
Contributor

@mannycarrera4 Don't you still need to pass transformOrigin={null} for this story?

FYI, master changes are going in w/ #623

@mannycarrera4
Copy link
Contributor Author

ah yeah, in that case, I can just delete all the transform origin stuff since that went into master, and pass null to the story

@jaclynjessup jaclynjessup removed the ready for review Code is ready for review label Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This issue is waiting for one or more issues to be closed/resolved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants