-
Notifications
You must be signed in to change notification settings - Fork 362
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
Seedlet Blocks: Missing WPcom-specific footer-filtering #2271
Comments
The "Proudly powered by WordPress" is in the footer.html block template part in the seedlet blocks theme: https://github.com/Automattic/themes/blob/master/seedlet-blocks/block-template-parts/footer.html#L6 Filters can't be run here, so I imagine the idea would be to provide a different footer block template part for WordPress.com? |
I almost think we wouldn't the "footer-credit" (let's call it that for short) in the template part? But instead always append it to the footer template part? Or perhaps, I suppose it could be a custom footer-credit block in the footer that was "locked" for all roles? Or had deletion alone locked? |
We've discussed adding custom footer credit blocks for wordpress.com. But yeah, the core functionality based on "can't change it without a plan" won't work without a custom block like that. There is also this issue about dynamic content in template parts which may see some progress in the coming months: WordPress/gutenberg#21932 |
Yeah, perhaps a custom block that was locked https://developer.wordpress.org/block-editor/developers/block-api/block-templates/#locking in some way? |
Funnily enough, I already have some basic foundation for this type of block from our dotcom-fse explorations: Automattic/wp-calypso#37012 |
This reminds me, because I pulled up an old FSE test site the other day, if we make these sorts of blocks (that is, if that's the solution) we should make sure they can't be broken in the code editor view of an Editor. (I found out I could break my FSE test site that way in some instances.) |
Yeah, totally. We haven't integrated code editor into the site editor yet because it's a tricky problem -- currently it's only designed to work with a single entity (post) and not a ton at the same time :/ |
We fixed this directly in the credits plugin for all block themes |
WPcom uses a plugin, filter, and boilerplate code in footer templates for traditional WordPress themes to add custom text to the footer of every WP.com site like the following example from a Varia-based theme:
This filtering is not currently happening with Seedlet-Blocks:
What I expected:
The same, "Create a free website or blog at WordPress.com." message in the footer of Seedlet Blocks.
The text was updated successfully, but these errors were encountered: