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

Added missing "Code is Poetry" block in some packages #13101

Merged
merged 8 commits into from
Jan 7, 2019
Merged

Added missing "Code is Poetry" block in some packages #13101

merged 8 commits into from
Jan 7, 2019

Conversation

mukeshpanchal27
Copy link
Member

Description

Some of the package documents of gutenberg handbook are missing "Code is Poetry" block at the end of the page.

@aduth aduth added the npm Packages Related to npm packages label Jan 4, 2019
@@ -27,3 +27,5 @@ count accepts three parameters:
```JS
import { count } from '@wordpress/wordcount';
const numberOfWords = count( 'Words to count', 'words', {} )

<br/><br/><p align="center"><img src="https://s.w.org/style/images/codeispoetry.png?1" alt="Code is Poetry." /></p>
Copy link
Member

Choose a reason for hiding this comment

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

This does not display correctly, as the prior code block is not terminated.

Copy link
Member Author

Choose a reason for hiding this comment

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

@aduth added end tag of JS "````"

Copy link
Member

Choose a reason for hiding this comment

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

There should only be three back ticks to end the block. It does seem to display correctly in the preview though, which is unexpected to me.

Copy link
Member Author

Choose a reason for hiding this comment

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

@aduth thanks for your guidance

Copy link
Member

@aduth aduth left a comment

Choose a reason for hiding this comment

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

Thanks 👍

@aduth aduth merged commit 4fe36cc into WordPress:master Jan 7, 2019
@mukeshpanchal27 mukeshpanchal27 deleted the documentation/add-missing-block-in-packages branch January 7, 2019 17:11
youknowriad pushed a commit that referenced this pull request Mar 6, 2019
* docs: added correct name for font site 50

* Remove old changes

* Added missing "Code is Poetry" block in some packages documents

* Revert "Remove old changes"

This reverts commit c7622d9.

* Revert "docs: added correct name for font site 50"

This reverts commit b30c96a.

* Fix display issue in wordcount package

* Fix end tag for js block
youknowriad pushed a commit that referenced this pull request Mar 6, 2019
* docs: added correct name for font site 50

* Remove old changes

* Added missing "Code is Poetry" block in some packages documents

* Revert "Remove old changes"

This reverts commit c7622d9.

* Revert "docs: added correct name for font site 50"

This reverts commit b30c96a.

* Fix display issue in wordcount package

* Fix end tag for js block
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
npm Packages Related to npm packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants