-
Notifications
You must be signed in to change notification settings - Fork 361
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
Conversation
There was a problem hiding this 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 🚢
@@ -0,0 +1,15 @@ | |||
<?php | |||
/** | |||
* Pattern Name: A 404 page |
There was a problem hiding this comment.
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...
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
Co-authored-by: Jeff Ong <[email protected]>
Co-authored-by: Jeff Ong <[email protected]>
Co-authored-by: Jeff Ong <[email protected]>
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.