-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
PanelColorSettings: add README.md file #17996
Conversation
@@ -0,0 +1,42 @@ | |||
# `PanelColorSettings` | |||
|
|||
Render a panel with color choices shown in circles, allowing a user to pick them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think a really important thing to mention here is that PanelColorSettings
can render multiple color palettes.
* *Required:* No | ||
* *Default:* Null | ||
|
||
The content to be displayed within the panel. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Specifically below the color palettes. It could also be worth mentioning that this is useful for rendering a ContrastChecker
within the panel.
* **Type:** `Array` | ||
* *Required:* Yes | ||
|
||
An array of props used for mapping both ColorIndicator and ColorPaletteControl components. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it might be good to mention that each entry in the array represents an individual color palette. An example showing the shape of the object might be good here as well.
|
||
### `disableCustomColors` | ||
|
||
Whether to allow custom color or not. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor nitpick:
Whether to allow custom color or not. | |
Whether to allow custom colors or not. |
Thanks for adding these docs @retrofox! I left a few minor comments on the PR where I think the descriptions can be improved. |
Description
It adds the README.md file to the
PanelColorSettings
compoment.How has this been tested?
Just read the doc.