Skip to content

added the allowedFormat details in richtext readme#42426

Merged
fabiankaegy merged 5 commits intoWordPress:trunkfrom
Smit2808:update/richtext-readme
Jul 29, 2022
Merged

added the allowedFormat details in richtext readme#42426
fabiankaegy merged 5 commits intoWordPress:trunkfrom
Smit2808:update/richtext-readme

Conversation

@Smit2808
Copy link
Copy Markdown
Contributor

What?

In this PR I have added the information about the allowedFormats property. In richtext package documentation, there is no detailed information and example about this property. So created the PR for this.

Why?

In rich-text package documentation, there should be detailed information about the allowedFormats property.

The allowedFormats property provides the formatting options in tools. If you want to display only limited formatting options to richtext then allowedFormats is used.

How?

In my custom Gutenberg block, I want to stop displaying some formating options so I researched a lot and found about this allowedFormats property in one blog. So I think there should be detailed example documentation about allowedFormats property in rich text package documentation.

Testing Instructions

Open the rich text documentation and you will not find any detailed info about allowedFormats property.

Screenshots or screencast

@github-actions github-actions bot added the First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository label Jul 14, 2022
@github-actions
Copy link
Copy Markdown

👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @Smit2808! In case you missed it, we'd love to have you join us in our Slack community, where we hold regularly weekly meetings open to anyone to coordinate with each other.

If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information.

Copy link
Copy Markdown
Member

@fabiankaegy fabiankaegy left a comment

Choose a reason for hiding this comment

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

Hey @Smit2808 👋

Thanks for creating this new PR and improving the documentation :)

I have left a few suggestions in the code. Mainly I think that we should remove all the code comments that don't have anything to do with the allowedFormats property as it makes the code example a little crowded.

Once these suggestions have been applied we can merge this in :)

@fabiankaegy fabiankaegy added [Type] Developer Documentation Documentation for developers [Package] Block editor /packages/block-editor labels Jul 29, 2022
Smit2808 and others added 4 commits July 29, 2022 11:59
Co-authored-by: Fabian Kägy <mail@fabian-kaegy.de>
Co-authored-by: Fabian Kägy <mail@fabian-kaegy.de>
Co-authored-by: Fabian Kägy <mail@fabian-kaegy.de>
Co-authored-by: Fabian Kägy <mail@fabian-kaegy.de>
@Smit2808
Copy link
Copy Markdown
Contributor Author

Smit2808 commented Jul 29, 2022

Hi @fabiankaegy

Thanks 🙂 for reviewing my PR. I committed all suggestions you have given. Please review it.

Copy link
Copy Markdown
Member

@fabiankaegy fabiankaegy left a comment

Choose a reason for hiding this comment

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

This is ready to get merged :)

(we'll have to wait for the checks to pass and then it can get merged in)

@Smit2808
Copy link
Copy Markdown
Contributor Author

Thanks, @fabiankaegy for the quick reply and review.

@Smit2808
Copy link
Copy Markdown
Contributor Author

Hi @fabiankaegy, All checks are passed. Now you can merge.

@fabiankaegy fabiankaegy merged commit 2688b3e into WordPress:trunk Jul 29, 2022
@github-actions github-actions bot added this to the Gutenberg 13.9 milestone Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository [Package] Block editor /packages/block-editor [Type] Developer Documentation Documentation for developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants