Skip to content

/ gutenberg

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

Jump to bottom

Fix DuotonePicker docs #34494

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from
Open

Fix DuotonePicker docs #34494

wants to merge 2 commits into from
+6 −6

Conversation

@madalingorbanescu
Copy link

@madalingorbanescu madalingorbanescu commented Sep 2, 2021

Description

Fixes the DuotonePicker docs.

  • Added missing , for DUOTONE_PALETTE objects properties.
  • Replaced COLOR_PALETTE colors property to color according to the API.
@madalingorbanescu
Fix DuotonePicker docs
67d2d3b
@github-actions
Copy link

@github-actions github-actions bot commented Sep 2, 2021

👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @madalingorbanescu! 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.

@ajlende
COLOR_PALETTE color should just be a string
5abf8f9
@ajlende
ajlende approved these changes Sep 2, 2021
View changes
Copy link
Contributor

@ajlende ajlende left a comment

Thanks for the fix! It looked like the color in the color palette should also just be a string, so I added a commit for that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajlende ajlende

@ajitbohra ajitbohra

@chrisvanpatten chrisvanpatten

Assignees
No one assigned
Labels
First-time Contributor [Type] Documentation
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@madalingorbanescu @ajlende @skorasaurus