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

Feature request: Display both default duotones and custom duotones #38690

Closed
carolinan opened this issue Feb 10, 2022 · 1 comment · Fixed by #38681
Closed

Feature request: Display both default duotones and custom duotones #38690

carolinan opened this issue Feb 10, 2022 · 1 comment · Fixed by #38681
Labels
[Feature] Design Tools Tools that impact the appearance of blocks both to expand the number of tools and improve the experi [Type] Enhancement A suggestion for improvement.

Comments

@carolinan
Copy link
Contributor

What problem does this address?

I received a support request asking why the default duotones are not available when you add custom duotones.
That the default duotones are not available when you add custom duotones is the opposite behavior of the color palette and the gradients, where you add a setting in theme.json if you want to hide them.

What is your proposed solution?

Always display the default duotones unless they are disabled.
Add a new setting to not display the default duotones to theme.json, marching "defaultPalette" and "defaultGradients" booleans.

@Mamaduka
Copy link
Member

The defaultDuotone PR - #38681.

@annezazu annezazu added [Feature] Design Tools Tools that impact the appearance of blocks both to expand the number of tools and improve the experi [Type] Enhancement A suggestion for improvement. labels Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Design Tools Tools that impact the appearance of blocks both to expand the number of tools and improve the experi [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants