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

Seedlet Blocks: Missing WPcom-specific footer-filtering #2271

Closed
ianstewart opened this issue Jul 20, 2020 · 8 comments
Closed

Seedlet Blocks: Missing WPcom-specific footer-filtering #2271

ianstewart opened this issue Jul 20, 2020 · 8 comments

Comments

@ianstewart
Copy link
Contributor

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:

image

This filtering is not currently happening with Seedlet-Blocks:

image

What I expected:

The same, "Create a free website or blog at WordPress.com." message in the footer of Seedlet Blocks.

@ianstewart ianstewart added the [Type] Bug Something isn't working label Jul 20, 2020
@apeatling
Copy link
Member

apeatling commented Jul 21, 2020

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?

@ianstewart
Copy link
Contributor Author

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?

@noahtallen
Copy link
Contributor

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

@ianstewart
Copy link
Contributor Author

ianstewart commented Aug 19, 2020

won't work without a custom block like that

Yeah, perhaps a custom block that was locked https://developer.wordpress.org/block-editor/developers/block-api/block-templates/#locking in some way?

@noahtallen
Copy link
Contributor

noahtallen commented Aug 19, 2020

Funnily enough, I already have some basic foundation for this type of block from our dotcom-fse explorations: Automattic/wp-calypso#37012

@ianstewart
Copy link
Contributor Author

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.)

@noahtallen
Copy link
Contributor

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 :/

@MaggieCabrera
Copy link
Contributor

We fixed this directly in the credits plugin for all block themes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants