Add news blog and archive query loop pattern#284
Conversation
…add newsletter footer part
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Unlinked AccountsThe following contributors have not linked their GitHub and WordPress.org accounts: @davidbp845@riseup.net. Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases. If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
Hey @dballari - I noticed the new footer-newsletter is not added to the theme.json - I think it should be added as a templatePart in theme.json |
|
Hi @KevinTaron yes you are right. Thanks a lot. I just added it. |
|
Why is the template part needed instead of just using the correct pattern directly? |
|
Ignore the question, I will assume it is for consistency with the other templates. |
Hi @carolinan in the begining I did not think of it, I just did it for consistency, but now I'm thinking: we should be able to create new templates from the editor, right? not just from the code, and then for instance if we want a template for a certain category it will be nice to be able to add the footer newsletter template part as a template part, no just a pattern right? Anyway, there is something I'm missing here, I do not now how to retrieve the news blog templates from the editor ... I have a lot to learn here. I will be working on this until I fully understand how it works |
|
I can help with that. From the Site editor menu, -the dark grey sidebar, go to Templates. While editing the template, you may first need to reset it: Because the editor does not actually show which pattern is currently used. Below the description is the Content panel. Skip this and open the next panel, Design. |
|
We have a design approval, the issue with the closing group block has been fixed, and all the templates have the footer updated, so lets merge this. |
Description
This PR solves the issue #275
Screenshots
Testing Instructions