Opened 19 months ago
Closed 18 months ago
#57827 closed defect (bug) (fixed)
Remove `wp-nux` CSS as dependency from `wp-edit-post` and `wp-editor`.
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Script Loader | Keywords: | |
Focuses: | performance | Cc: |
Description
The CSS for both wp-edit-post
and wp-editor
list the wp-nux
CSS file as a dependency. This results in nux.css
being loaded when using these packages.
Neither wp-edit-post
or wp-editor
use any of the styles included in nux.css
so the dependency can be removed.
Related #57643.
A pull request exists for this. I'll see if I can get it to link to this ticket too.
Change History (5)
#2
in reply to:
↑ 1
@
19 months ago
Replying to flixos90:
Thanks @peterwilsoncc, the change makes sense to me. Just wondering, is this really a bug? If it is, maybe we can still commit it for 6.2 Beta.
I think it's one of those items that falls neatly between bug and enhancement. I lean toward the former but left the ticket in review for the release squad to decide if it's the latter.
I'm happy with whatever milestone you decide.
Thanks @peterwilsoncc, the change makes sense to me. Just wondering, is this really a bug? If it is, maybe we can still commit it for 6.2 Beta. But to me removing a dependency that's unused seems more like an enhancement, unless it was causing any actual bugs for the end user. In that case we could commit it once trunk opens for 6.3.