Skip to content
This repository has been archived by the owner on Oct 28, 2022. It is now read-only.

Variation "Marigold" #137

Closed
nudgeD opened this issue Aug 31, 2022 · 1 comment · Fixed by #156
Closed

Variation "Marigold" #137

nudgeD opened this issue Aug 31, 2022 · 1 comment · Fixed by #156

Comments

@nudgeD
Copy link

nudgeD commented Aug 31, 2022

Variation-colors

Homepage

Colors

Buttons and Links

Figma:
https://www.figma.com/file/GQh990IZ2EaBYZhU93hdi1/TT3%3A-Marigold-Style-Variation?node-id=455%3A426

JSON:
marigold.json.zip

@critterverse
Copy link
Collaborator

This style variation has been selected to be bundled with the Twenty Twenty-Three default theme! @ beafialho and I have made some tweaks to the submission as a part of the curation process to ensure that all the selected variations work together nicely as a collection. Here are the requested changes:

Marigold

  • We weren’t sure about the use of DM Sans here, which doesn’t have a light weight and feels a bit round/heavy next to the more delicate Source Serif typeface. Instead, this submission could take some inspiration from @melchoyce's Soft variation and use Source Serif throughout for both headlines and paragraph text.

No explicit next steps are needed — feel free to update theme.json files where relevant or they will be updated in development. Thanks for the contribution!

bobmatyas added a commit to bobmatyas/twentytwentythree that referenced this issue Sep 11, 2022
@mikachan mikachan linked a pull request Sep 17, 2022 that will close this issue
mikachan added a commit that referenced this issue Sep 19, 2022
* #137: Add Marigold style variation

* Match spacing size names to base theme

* Remove duplicate theme.json entries

* Change all fonts to source serif pro

Co-authored-by: Sarah Norris <sarah@sekai.co.uk>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants