[wp-trac] [WordPress Trac] #56990: Render blocking CSS `classic-themes.css` unnecessarily enqueued
WordPress Trac
noreply at wordpress.org
Mon Feb 6 21:59:52 UTC 2023
#56990: Render blocking CSS `classic-themes.css` unnecessarily enqueued
-------------------------------------+-------------------------------
Reporter: adamsilverstein | Owner: flixos90
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 6.2
Component: Themes | Version: 6.1
Severity: normal | Resolution:
Keywords: has-patch needs-testing | Focuses: css, performance
-------------------------------------+-------------------------------
Changes (by flixos90):
* owner: (none) => flixos90
* status: new => assigned
Comment:
@scruffian @adamsilverstein @wpgurudev I have opened
https://github.com/WordPress/wordpress-develop/pull/4012, which I believe
is a simpler fix to the most prevalent performance issue here, and it is a
backward compatible change, which I think makes this a better solution
than the previous patches. See the PR description for more context. Can
you please take a look?
I acknowledge that the perfect outcome here would be to not even load this
CSS if no buttons are being used on the page, but as you are already
saying @adamsilverstein, this is challenging to reliable determine in a
classic theme. Since the amount of CSS here is tiny though, I don't really
see that itself as a performance concern. The main problem is that the CSS
is loaded via an external file, which adds unnecessary overhead that we
can avoid by inlining it.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/56990#comment:35>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list