Skip to content

Try different block descriptions#10971

Merged
youknowriad merged 3 commits intomasterfrom
try/block-description-edits
Oct 30, 2018
Merged

Try different block descriptions#10971
youknowriad merged 3 commits intomasterfrom
try/block-description-edits

Conversation

@karmatosed
Copy link
Copy Markdown
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.

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
Copy Markdown
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
Copy Markdown
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.

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

title: __( 'Spacer' ),

description: __( 'Add an element with empty space and custom height.' ),
description: __( 'Add white space between blocks and customize its height.' ),
Copy link
Copy Markdown
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)?

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
Copy Markdown
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.

title: __( 'Image' ),

description: __( 'They’re worth 1,000 words! Insert a single image.' ),
description: __( 'Insert an image for visual color.' ),
Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

@youknowriad
Copy link
Copy Markdown
Contributor

For information, I'm going to update the PR

@youknowriad youknowriad force-pushed the try/block-description-edits branch from 2c5e12b to 1ed4aa7 Compare October 30, 2018 09:08
@youknowriad
Copy link
Copy Markdown
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
Copy Markdown
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.

@youknowriad youknowriad force-pushed the try/block-description-edits branch from 1ed4aa7 to a97364a Compare October 30, 2018 10:28
@youknowriad youknowriad force-pushed the try/block-description-edits branch from a97364a to 433f128 Compare October 30, 2018 11:51
@karmatosed
Copy link
Copy Markdown
Member Author

Thanks for the iterations @youknowriad.

@youknowriad
Copy link
Copy Markdown
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