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

Create Pendant Base Theme #5694

Merged
merged 4 commits into from
Mar 17, 2022
Merged

Create Pendant Base Theme #5694

merged 4 commits into from
Mar 17, 2022

Conversation

pbking
Copy link
Contributor

@pbking pbking commented Mar 16, 2022

Cloned 'Block Canvas' via the Create Block Theme plugin and adjusted the palette colors as a starting point for Pendant.

This closes #5683
This also closes #5688

Currently this looks a LOT like Blockbase. That is expected.

image

image

@pbking pbking requested a review from a team March 16, 2022 18:22
Copy link
Contributor

@jffng jffng left a comment

Choose a reason for hiding this comment

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

Couple nits but I think this is good to bring in and iterate, awesome that it was bootstrapped by create block theme 🚢

pendant/functions.php Outdated Show resolved Hide resolved
pendant/functions.php Outdated Show resolved Hide resolved
pendant/functions.php Outdated Show resolved Hide resolved
@@ -0,0 +1,15 @@
<?php
/**
* Pattern Name: A 404 page
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't register this so an empty pattern shows up in the site editor. I think it's fine to do this in a follow-up, unless you wanted to do it now...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We're still waiting for this to land at which point this should start working. It doesn't work now but I hope it will before we start in on patterns (and the 404 page). This was copied from @MaggieCabrera 's new boilerplate theme.

My 'druthers is to bring it in now.

Copy link
Contributor

Choose a reason for hiding this comment

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

The pattern PR in GB is going to be merged soon, and it most likely will make it to dotcom before the theme is ready to launch, but we can of course make a patch in the meantime.

@pbking pbking merged commit d4929cb into trunk Mar 17, 2022
@scruffian scruffian deleted the add/pendant branch March 17, 2022 16:33
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.

Pendant: Global Styles: Color palette Pendant: Create base theme
3 participants