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 page templates button alignment and strings #2314

Merged
merged 10 commits into from
May 26, 2020

Conversation

mkevins
Copy link
Contributor

@mkevins mkevins commented May 26, 2020

Fixes: #2287 #2295 #2267

Related PR:

gutenberg: WordPress/gutenberg#22620

Description

This PR fixes some strings in the mobile pages templates to be consistent, and also uses center alignment for the buttons.

To test:

In a new page, apply each of the templates and verify that the buttons are center aligned, and that the following strings are consistent "Let's build something together!" (should have an exclamation mark at the end), and "Get in Touch" (consistently title case for each instance).

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@mkevins mkevins added the bugfix label May 26, 2020
@mkevins mkevins requested a review from geriux May 26, 2020 07:38
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented May 26, 2020

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

Copy link
Contributor

@geriux geriux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mkevins
Copy link
Contributor Author

mkevins commented May 26, 2020

Thank you again for testing / reviewing Gerardo!

@mkevins mkevins merged commit e01a976 into develop May 26, 2020
@mkevins mkevins deleted the fix/page-templates-button-alignment-and-strings branch May 26, 2020 10:30
@mchowning mchowning mentioned this pull request May 26, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants