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 intro CYS logo and site title #45216
Conversation
eb4b475
to
13aebcd
Compare
Apart from reviewing the code changes, please make sure to review the testing instructions as well. You can follow this guide to find out what good testing instructions should look like: |
Test Results SummaryCommit SHA: 07de2c1
To view the full API test report, click here. To view the full E2E test report, click here. To view all test reports, visit the WooCommerce Test Reports Dashboard. |
Thanks for working on this! It doesn't look like a regression introduced in this PR, but I noticed that during the transition between the assembler hub and the intro page, the logo has a weird animation: Screen.Capture.Feb.29.at.16-11.mp4Given that this PR is smaller, it could make sense to include the fix in this PR. WDYT? |
@gigitux Thanks for the feedback, I've applied the changes you mentioned on Slack. |
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.
LGTM and tested well. 🚢
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.
LGTM!
* Fix intro CYS logo and site title * Add changefile(s) from automation for the following project(s): woocommerce * Improve format * Fix logo motion --------- Co-authored-by: github-actions <github-actions@github.com>
* Fix intro CYS logo and site title * Add changefile(s) from automation for the following project(s): woocommerce * Improve format * Fix logo motion --------- Co-authored-by: github-actions <github-actions@github.com>
Submission Review Guidelines:
Changes proposed in this Pull Request:
This PR fixes a CSS issue with the CYS intro page logo and site title.
How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
WooCommerce Beta Tester
plugin is installed and activated (available on this monorepo)./wp-admin/tools.php?page=woocommerce-admin-test-helper
and enablecustomize-store
feature flag.Tools
and run theReset Customize Your Store
and theDelete all products
commands.WooCommerce > Home
, click onStart customizing
.After
screenshot below:Changelog entry
Significance
Type
Message
CYS - Fix the intro page logo and site title positioning.
Comment