Skip to content

Added style variations in Purr Theme#223

Open
shail-mehta wants to merge 8 commits into
WordPress:trunkfrom
shail-mehta:add-style-variation
Open

Added style variations in Purr Theme#223
shail-mehta wants to merge 8 commits into
WordPress:trunkfrom
shail-mehta:add-style-variation

Conversation

@shail-mehta
Copy link
Copy Markdown
Member

  • I have added One Style Variation(verdigris).
  • Changed Color palette name in theme.json (Used base, base-2, contrast , contrast-2, contrast-3, accent, accent-2 Instead of particular color name for Reuse name in custom style variation)
  • Added Font License Info in readme.txt
  • Changed required php version in style.css (added 7.4 same as readme.txt)
  • Changed Tested Up to 6.6 in style.css & readme.txt (Because we are already in 6.6 RC1 was already released)

@shail-mehta shail-mehta changed the title Added style variations (verdigris) in Purr Theme Added style variation (verdigris) in Purr Theme Jun 29, 2024
@shail-mehta
Copy link
Copy Markdown
Member Author

shail-mehta commented Jun 29, 2024

Issue: #200

Comment thread purr/theme.json
@shail-mehta shail-mehta requested a review from MaggieCabrera July 7, 2024 07:37
@shail-mehta shail-mehta changed the title Added style variation (verdigris) in Purr Theme Added style variations in Purr Theme Jul 13, 2024
@shail-mehta
Copy link
Copy Markdown
Member Author

shail-mehta commented Jul 13, 2024

Added One more Style Variation. (twilight Style Variation)

"fontStyle": "normal",
"fontWeight": "700",
"src": [
"file:./assets/fonts/dm-sans/DMSans-Bold.woff2"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm not sure it's a bug with your code but I don't see DM sans on the editor. I do see it applied in the frontend, though

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Its working for me in both editor as well as frontend too.

1. Editor:

editor

2. Front-end:

frontend

Comment thread purr/styles/verdigris.json Outdated
@shail-mehta
Copy link
Copy Markdown
Member Author

Okay, I will Check both points again.

@shail-mehta
Copy link
Copy Markdown
Member Author

@MaggieCabrera I have Fixed Color Contrast Issue in Latest Commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants