Make WordPress Core

Opened 4 years ago

Closed 3 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)

#1 follow-up: @flixos90
4 years ago

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.

Last edited 4 years ago by flixos90 (previous) (diff)

#2 in reply to: ↑ 1 @peterwilsoncc
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.

#3 @peterwilsoncc
3 years ago

  • Milestone Awaiting Review6.3

#4 @peterwilsoncc
3 years ago

  • Component EditorScript Loader

#5 @peterwilsoncc
3 years ago

  • Owner set to peterwilsoncc
  • Resolutionfixed
  • Status newclosed

In 55628:

Script Loader: Remove unused wp-nux CSS dependency.

Neither wp-edit-post nor wp-editor use the styles included in the wp-nux CSS file. This removes nux as a dependency from the registration of the two files.

Props flixos90, youknowriad, peterwilsoncc.
Fixes #57827.
See #57643.

Note: See TracTickets for help on using tickets.