Skip to content

Conversation

@sabernhardt
Copy link
Contributor

The end quote for the class attribute was touching href, and Firefox highlighted the markup as invalid.

Note: the typo is already in WordPress 5.9 trunk, so this would need fixing in both places.

@talldan talldan added [Block] Page List Affects the Page List Block [Type] Bug An existing feature does not function as intended Backport to WP 6.9 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta labels Nov 16, 2021
Copy link
Contributor

@talldan talldan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for fixing!

I think the block PHP files are copied over to core when backporting, so not sure if this requires a separate change, or whether it will automatically propagate over in the next beta release. cc @noisysocks

@talldan talldan merged commit e1d8364 into WordPress:trunk Nov 16, 2021
@github-actions github-actions bot added this to the Gutenberg 12.0 milestone Nov 16, 2021
@noisysocks
Copy link
Member

It will happen automatically as it's part of the block PHP.

@noisysocks noisysocks removed the Backport to WP 6.9 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 22, 2021
@sabernhardt sabernhardt deleted the patch-1 branch May 10, 2024 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Block] Page List Affects the Page List Block [Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants