Greg Ziółkowski
Greg Ziółkowski
> That ticket might be a bit broader in scope than just this PR, since there's strictly speaking no need for bookmark invalidation as long as there isn't any API...
> Also, there's an experiment running to stop loading TinyMCE when possible, see #50387. We could explore having a similar mechanism for loading scripts handling meta boxes only when they...
I rebase this branch with `trunk`. Refactored code to use the new class name `WP_HTML_Tag_Processor`. One of the core blocks Post Featured Image got refactored and it no longer is...
> Esc_attr shouldn’t be needed thanks to #44447. Let’s remove it to avoid escaping the value twice. Nice, it still might be necessary to use some sanitization in some cases...
@adamziel and @dmsnell, I think this PR depends on how https://github.com/WordPress/gutenberg/pull/44447 ends up handling escaping attributes. It would be great if you could bring it to the finish line. [1983b8f](https://github.com/WordPress/gutenberg/pull/43178/commits/1983b8fdb8f3e0b29cf2848ffb0bd97201a96948)...
I refreshed this PR to use the latest API. It doesn't matter if we include it in WordPress 6.2. Actually, it's probably better to test it in the plugin first.
## Testing ### Cover block ✅ Minimum height style applied correctly ### Image block ✅ Aria label and target applied correctly ### Social Icon ✅ Still properly handles `rel` and...
> I'd like to know if hurry was the only reason [the regex for removing the link from the site logo](https://github.com/WordPress/gutenberg/blob/trunk/packages/block-library/src/site-logo/index.php#L36) was left as is? HTML Tag Processor doesn't support...
@audrasjb, it looks like you need to rebase your fork with the `trunk` from the upstream repository. As far as I can tell, all descriptions end with the period after...
> Until now, I can't remember having any problems with the build script in @wordpress/scripts: It built both the blocks and the components found in the /src directory. But now,...