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.

Sign up for GitHub

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

Jump to bottom

Use "homepage" instead of "home page" #11458

Closed
Presskopp opened this issue Nov 3, 2018 · 2 comments
Closed

Use "homepage" instead of "home page" #11458

Presskopp opened this issue Nov 3, 2018 · 2 comments
Assignees
@Soean
Labels
[Package] Block library /packages/block-library [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Copy Issues or PRs that need copy editing assistance

Comments

@Presskopp
Copy link
Contributor

Presskopp commented Nov 3, 2018 •
edited by ocean90
Loading

We usually use "homepage", not "home page" in strings

https://github.com/WordPress/gutenberg/blob/db728b7/packages/block-library/src/more/index.js#L28

Want to show only an excerpt of this post on your home page? Use this block to define where you want the separation.

Also I find this sentence a bit 'unpolished'. I'd expect more st. like

Do you want to show only an excerpt of this post on your homepage? Then use this block to define where the separation should take place.

related: https://core.trac.wordpress.org/ticket/42468
@swissspidy
Copy link
Member

For reference, the core handbook glossary: https://make.wordpress.org/core/handbook/best-practices/spelling/

@ocean90 ocean90 added [Type] Copy Issues or PRs that need copy editing assistance [Package] Block library /packages/block-library Needs Copy Review Needs review of user-facing copy (language, phrasing) labels Nov 4, 2018
@benhuberman
Copy link

We usually use "homepage", not "home page" in strings

That's a good catch -- it would need to be applied to a couple more instances in #10919. cc @alexislloyd.

@Soean Soean self-assigned this Nov 7, 2018
@desrosj desrosj added the [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later label Nov 7, 2018
@Soean Soean mentioned this issue Nov 7, 2018
Merged
@Soean Soean removed the Needs Copy Review Needs review of user-facing copy (language, phrasing) label Nov 7, 2018
@gziolo gziolo mentioned this issue Nov 9, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Soean Soean

Labels
[Package] Block library /packages/block-library [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Copy Issues or PRs that need copy editing assistance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@desrosj @ocean90 @Soean @swissspidy @benhuberman @Presskopp