Remove Fetchpriority module as the functionality is now available in WordPress core#854
Conversation
|
I understand that the rationale here is that since fetchpriority has landed in WP 6.3, that this plugin is essentially finished software and could be archived. However, we continue to have ongoing conversations about how to improve the heuristics and markup techniques used to accurately apply fetchpriority to the best LCP candidate. For that reason, I wonder if we should keep this plugin in active development as a way of quickly iterating on those improvements prior to them being shipped in an official WP release? |
|
@joemcgill I thought about this as well before agreeing to remove the module, my conclusion is the following: Yes, we may want to improve the heuristics, but to me this does not fit into this existing module / plugin, as its goal was to enable fetchpriority support. Improving the heuristics affects multiple attributes ( |
felixarntz
left a comment
There was a problem hiding this comment.
Thanks @mukeshpanchal27, LGTM!
I had the very same thought when writing my original comment. I'm onboard with that plan. |
Summary
Fixes #853
Checklist
[Focus]orInfrastructurelabel.[Type]label.no milestonelabel.