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

HTML API: Pull updates from development in Core #48378

Merged
merged 1 commit into from
Feb 24, 2023

Conversation

dmsnell
Copy link
Contributor

@dmsnell dmsnell commented Feb 23, 2023

This commit pulls in updates from development of the HTML API happening in Core.

  • HTML API: Fix finding bookmarks set on closing tag WP_HTML_Tag_Processor
    83ae6b790e5f1d361b5b128063f74bf253b55397

  • HTML API: Set $this->html to protected to support subclassing
    552178a95ad38c924f9d9821d3c75d0bf9926748

Created by copying tag processor class into compat/wordpress-6.3 and renaming the class.

@github-actions
Copy link

github-actions bot commented Feb 23, 2023

Flaky tests detected in 79a5406aea243fca05f9a50be758adf79975f7e2.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4258643113
📝 Reported issues:

@dmsnell dmsnell force-pushed the html-api/updates-from-core-dev branch from e23dddd to 79a5406 Compare February 24, 2023 02:06
Copy link
Contributor

@adamziel adamziel left a comment

Choose a reason for hiding this comment

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

Awesome, thank you!

This commit pulls in updates from development of the HTML API happening in Core.

 - HTML API: Fix finding bookmarks set on closing tag WP_HTML_Tag_Processor
   83ae6b790e5f1d361b5b128063f74bf253b55397

 - HTML API: Set $this->html to protected to support subclassing
   552178a95ad38c924f9d9821d3c75d0bf9926748
@dmsnell dmsnell force-pushed the html-api/updates-from-core-dev branch from 79a5406 to c846c62 Compare February 24, 2023 23:00
@dmsnell dmsnell merged commit c846c62 into trunk Feb 24, 2023
@dmsnell dmsnell deleted the html-api/updates-from-core-dev branch February 24, 2023 23:48
@ockham
Copy link
Contributor

ockham commented Mar 2, 2023

Since those changes will be in WP 6.2, we also need to backport them into Gutenberg's 6.2 compat layer. This is done in

ockham added a commit that referenced this pull request Mar 2, 2023
In the 6.2 compat layer, support bookmarks pointing to closing tags.

This is for parity with Core in WP 6.2, see WordPress/wordpress-develop#4115 and [r55407](https://core.trac.wordpress.org/changeset/55407).

Note that these changes have already been backported to GB's 6.**3** compat layer in #48378. However, since the change will be in WP 6.2, we also have to apply it to GB's 6.2 compat layer.
ockham added a commit that referenced this pull request Mar 2, 2023
In the 6.2 compat layer, support bookmarks pointing to closing tags.

This is for parity with Core in WP 6.2, see WordPress/wordpress-develop#4115 and [r55407](https://core.trac.wordpress.org/changeset/55407).

Note that these changes have already been backported to GB's 6.**3** compat layer in #48378. However, since the change will be in WP 6.2, we also have to apply it to GB's 6.2 compat layer.
@dmsnell
Copy link
Contributor Author

dmsnell commented Mar 2, 2023

Thanks @ockham! - sorry I didn't think about that.

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.

None yet

3 participants