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

Add support for block themes #7

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

mikachan
Copy link

This PR tries out adding support for block themes.

It checks if the current theme supports block-templates, and if so, uses the template-canvas.php as the main template.

This solution is based on this comment.

To test, try enabling bbPress with the Twenty Twenty-Two or Twenty Twenty-Three themes. Without this PR, the forum pages will not render. With this PR, they should render correctly.

Trac ticket: https://bbpress.trac.wordpress.org/ticket/3487

@mikachan mikachan changed the title Try adding support for block themes Add support for block themes Dec 20, 2022
Copy link

@arafatjamil01 arafatjamil01 left a comment

Choose a reason for hiding this comment

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

Please add an extra * at the beginning of your comments. That will maintain consistency with the other comments in the file.

The code is working perfectly fine.

@mikachan
Copy link
Author

Thanks for the review, @arafatjamil01!

I've fixed up the comment formatting so it's consistent with the other comments in the file.

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.

2 participants