Skip to content
This repository was archived by the owner on Nov 18, 2024. It is now read-only.

Add optional color palettes - #107

Merged
carolinan merged 5 commits into
WordPress:trunkfrom
efu98:add-optional-color-palettes
Aug 27, 2024
Merged

carolinan merged 5 commits into
WordPress:trunkfrom
efu98:add-optional-color-palettes

Conversation

@efu98

@efu98 efu98 commented Aug 22, 2024 •

Copy link
Copy Markdown
Contributor

Resolves #24
Added optional color palettes to the theme

Screenshots
Capture d’écran 2024-08-26 à 23 24 10

Testing Instructions

  1. Activate the theme
  2. Customize theme
  3. Go to styles
  4. There should be 9 different palettes

@github-actions

github-actions Bot commented Aug 22, 2024 •

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: efu98 <elfu98@git.wordpress.org>
Co-authored-by: carolinan <poena@git.wordpress.org>
Co-authored-by: juanfra <juanfra@git.wordpress.org>
Co-authored-by: beafialho <beafialho@git.wordpress.org>
Co-authored-by: richtabor <richtabor@git.wordpress.org>
Co-authored-by: karmatosed <karmatosed@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@carolinan

Copy link
Copy Markdown
Contributor

Hi, can you pull the latest changes from trunk, so that the color palette can be tested easier with the templates?

@carolinan

Copy link
Copy Markdown
Contributor

The preview of the palette in the option only shows five colors, so the palettes look very similar.
If you move the accent colors before primary, the accent colors will show in the option.

@efu98

efu98 commented Aug 26, 2024

Copy link
Copy Markdown
Contributor Author

@carolinan Hello, i have merged trunk into the PR and re-ordered the colours :)

@carolinan

Copy link
Copy Markdown
Contributor

I saw your question, but lets merge this so that it can be tested and used, and iterate when or if needed.
Thank you.

@carolinan
carolinan merged commit 0696335 into WordPress:trunk Aug 27, 2024
@efu98
efu98 deleted the add-optional-color-palettes branch August 27, 2024 16:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add default and optional color palettes

3 participants