Opened 4 years ago
Closed 4 years ago
#57827 closed defect (bug) (fixed)
Remove `wp-nux` CSS as dependency from `wp-edit-post` and `wp-editor`.
| Reported by: | peterwilsoncc | Owned by: | peterwilsoncc |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.3 |
| Component: | Script Loader | Version: | |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: | performance |
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
@
4 years 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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.