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: correct spacing around AutomateWoo opt-in #1975

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

laurelfulford
Copy link
Contributor

All Submissions:

Changes proposed in this Pull Request:

This PR fixes some spacing around the AutomateWoo opt-in checkbox.

See 1207817176293825-as-1208851280188397

How to test the changes in this Pull Request:

  1. Install and activate AutomateWoo (a premium plugin).
  2. Navigate to WP Admin > AutomateWoo > Settings, and switch the Marketing Opt-In to "Opt In" at the top:

CleanShot 2024-11-26 at 16 46 14

  1. Run through a checkout and note the spacing on the checkbox on the second screen.

CleanShot 2024-11-26 at 16 48 31

  1. Apply this PR and run npm run build.
  2. Repeat step 3 and confirm the spacing looks okay:

CleanShot 2024-11-26 at 16 47 31

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

@laurelfulford
Copy link
Contributor Author

Thanks @leogermani!

@laurelfulford laurelfulford merged commit b10c4f9 into epic/ras-acc Nov 28, 2024
10 checks passed
@laurelfulford laurelfulford deleted the fix/automate-woo-optin branch November 28, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants