Skip to content

Change shops to shop's as it needs an apostrophe - #2138

Open
damiencarbery wants to merge 2 commits into
woocommerce:trunkfrom
damiencarbery:patch-1
Open

Change shops to shop's as it needs an apostrophe#2138
damiencarbery wants to merge 2 commits into
woocommerce:trunkfrom
damiencarbery:patch-1

Conversation

@damiencarbery

Copy link
Copy Markdown
Contributor

"shops appearance" - possessive so should be "shop's appearance". Or drop the s: "shop appearance" and avoid any questions about apostrophes.

"shops appearance" - possessive so should be "shop's appearance".
Or drop the s: "shop appearance" and avoid any questions about apostrophes.

@Aljullu Aljullu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for opening this PR, @damiencarbery! Good catch with the typo. I suggest using double quotes " to wrap the string, this way we don't need to escape the single quote '. I left the code suggestion inline below. 👇


<p>
<?php esc_html_e( 'Quickly and easily transform your shops appearance with Storefront child themes.', 'storefront' ); ?>
<?php esc_html_e( 'Quickly and easily transform your shop\'s appearance with Storefront child themes.', 'storefront' ); ?>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
<?php esc_html_e( 'Quickly and easily transform your shop\'s appearance with Storefront child themes.', 'storefront' ); ?>
<?php esc_html_e( "Quickly and easily transform your shop's appearance with Storefront child themes.", 'storefront' ); ?>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants