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

Try different block descriptions #10971

Merged
merged 3 commits into from
Oct 30, 2018
Merged

Conversation

karmatosed
Copy link
Member

This follows the latest suggestions by @alexislloyd in #10919.

The section it doesn't do is this:

If the content format has its own noun, then use "Embed a tweet", "embed a Facebook post", etc. If not, then default to "Embed Kickstarter content", "embed Mixcloud content", etc.

I think this needs some dev help to make happen. @mtias looping you in just incase here.

@@ -10,7 +10,7 @@ export const name = 'core/preformatted';
export const settings = {
title: __( 'Preformatted' ),

description: __( 'Add text that respects your spacing and tabs, and also allows styling.' ),
description: __( 'Add custom HTML code and preview it as you edit.' ),
Copy link
Member

Choose a reason for hiding this comment

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

This doesn't make sense for the Preformatted block. There is no "preview" mode; the Preformatted block is just for preformatted text.

Copy link
Contributor

Choose a reason for hiding this comment

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

Good point. This seems like it wasn't updated to the latest suggestion in this comment:

Add text that respects your spacing and tabs, and also allows styling.

@@ -9,7 +9,7 @@ export const name = 'core/separator';
export const settings = {
title: __( 'Separator' ),

description: __( 'Insert a horizontal line where you want to create a break between ideas.' ),
description: __( 'Create a break between ideas or sections with a horizontal line.' ),
Copy link
Member

Choose a reason for hiding this comment

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

It's not always a line. Even a core style uses 3 dots.

Copy link
Contributor

Choose a reason for hiding this comment

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

Good point. How about:

Create a break between ideas or sections with a horizontal separator.

Copy link
Contributor

@youknowriad youknowriad Oct 30, 2018

Choose a reason for hiding this comment

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

The gutenberg-mobile subfolder tricked me, I updated more descriptions but in the wrong folder :P

@@ -17,7 +17,7 @@ export const name = 'core/spacer';
export const settings = {
title: __( 'Spacer' ),

description: __( 'Add an element with empty space and custom height.' ),
description: __( 'Add white space between blocks and customize its height.' ),
Copy link
Member

Choose a reason for hiding this comment

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

Wouldn't "empty space" be just a bit clearer (and easier to translate)?

@@ -15,7 +15,7 @@ export const name = 'core/verse';
export const settings = {
title: __( 'Verse' ),

description: __( 'A block for haiku? Why not? Blocks for all the things! (See what we did here?)' ),
description: __( 'Insert haiku. Use special spacing formats. Or quote song lyrics. (See what we did here?)' ),
Copy link
Member

Choose a reason for hiding this comment

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

This is not a valid haiku, as far as I can tell. The first line has 4 syllables, but the third line has 5 syllables. They should both have the same number of syllables.

@@ -96,7 +96,7 @@ const schema = {
export const settings = {
title: __( 'Image' ),

description: __( 'They’re worth 1,000 words! Insert a single image.' ),
description: __( 'Insert an image for visual color.' ),
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this description be a bit more like the Video one? Same goes for the Audio description.

@youknowriad
Copy link
Contributor

Can we get this rebased and refreshed with the latest descriptions in #10919 to include in 4.2?

@youknowriad
Copy link
Contributor

For information, I'm going to update the PR

@youknowriad
Copy link
Contributor

youknowriad commented Oct 30, 2018

Pushed an update. How does it look now?

@youknowriad youknowriad added this to the 4.2 milestone Oct 30, 2018
@jasmussen
Copy link
Contributor

I left a few comments based on Zebs thoughts, let's get those in and ship, then we can enhance further as we go.

@karmatosed
Copy link
Member Author

Thanks for the iterations @youknowriad.

@youknowriad
Copy link
Contributor

Looks like there was a PR that got merged without upgrading package-lock.json I'm sneaking a small fix in this unrelated PR to make sure unit tests pass.

@youknowriad youknowriad merged commit db728b7 into master Oct 30, 2018
@youknowriad youknowriad deleted the try/block-description-edits branch October 30, 2018 12:35
@mtias mtias added [Type] Enhancement A suggestion for improvement. [Feature] Blocks Overall functionality of blocks [Type] Copy Issues or PRs that need copy editing assistance labels Oct 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Blocks Overall functionality of blocks [Type] Copy Issues or PRs that need copy editing assistance [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants