Add Blue Note #19
Add Blue Note #19
Conversation
| Description: | ||
| Requires at least: 6.1 | ||
| Tested up to: 6.2 | ||
| Requires PHP: 8.0 |
There was a problem hiding this comment.
why 8.0? are we using something specific to have that high a requirement?
There was a problem hiding this comment.
No, but we should encourage users to use a newer PHP version. Consider that the current version of PHP is 8.2.
There was a problem hiding this comment.
TIL: Create block theme does not support PHP 8, so this should be lowered until the plugin is updated 😞
There was a problem hiding this comment.
My dev env has wp_debug enabled and with PHP 8 I get a ton of warnings that are coming from core. I think in general going with 7 for now should be fine until core updates. In any case, CBT should update as well, I believe I saw an issue raised in their repo about it
blue-note/templates/home.html
Outdated
| <h2 class="wp-block-heading">By Topic</h2> | ||
| <!-- /wp:heading --> | ||
|
|
||
| <!-- wp:query {"queryId":0,"query":{"perPage":3,"pages":0,"offset":0,"postType":"post","order":"desc","orderBy":"date","author":"","search":"","exclude":[],"sticky":"","inherit":true},"displayLayout":{"type":"flex","columns":3},"align":"full","layout":{"type":"default"}} --> |
There was a problem hiding this comment.
These are not really posts, but categories. How should we show these?
There was a problem hiding this comment.
I think the archive title and the archive description only work on the archive template.
Maybe it is an advanced variation of a category block? Untested.
There was a problem hiding this comment.
WordPress/gutenberg#49094 I think this is what we want
|
Thanks so much for having a go at this! I will do a more thorough review of this on Monday, but it's looking like a great start! |
|
What is needed in order to commit this as a starting point? |
There was a problem hiding this comment.
I think this is a great starting point, thank you so much @carolinan for getting this one started! Lets open issues and work on the templates and patterns on separate PRs now
Add a rough start for Blue Note.
What's missing:
Known issues:
The contrast and primary color has the same color code