Jonny Harris

Results 164 comments of Jonny Harris

As this can be easily achieved via plugins ( drop-ins ), is there a place for this core or should we just close this ticket.

A good start might be to create a new / refreshed PR / patch for core. There will increase the chances of getting this into core.

I will have to look into this. This is not a simple fix and I need to getting up to speed. I am busy with other things at the moment....

Thanks for the feedback @peterwilsoncc . Actioned it here https://github.com/WordPress/wordpress-develop/pull/6393/commits/55026bcad3380ec1ee30e95379ba47be3c84994a

Committed in https://core.trac.wordpress.org/changeset/58122

Run some profiles, before and after, the results, are sadly not promising. ### 2023 theme #### Before ![Screenshot 2023-03-01 at 10 31 16](https://user-images.githubusercontent.com/237508/222114401-493b3274-ef2e-409f-a5e4-899507c628e6.png) #### After ![Screenshot 2023-03-01 at 10 31...

This file will need to be broken up into multiple files. https://github.com/WordPress/WordPress/blob/10838cb5a009a95de13fa4d2f50afabcb7ca26c7/wp-includes/class-wp-block-parser.php

@kasparsd Not sure how you are getting those numbers. I just run it locally again, 300 times, using the core performance teams tool. Running 300 times. Trunk Response Time (median),389.06...

@aristath I think I worked out what the performance regression is! Before, simplePie was conditionally loaded. [See](https://github.com/WordPress/wordpress-develop/blob/7130af403fff492d4bde3e2fbd08bc3d25150dfb/src/wp-includes/class-feed.php#L12-L14). Now it is always loaded. See Simplepie is only loaded in `fetch_feed`. Maybe...

@aristath That change was big a deal. See profiles Trunk This PR.