WordPress.org

Make WordPress Core

Opened 10 months ago

Last modified 2 months ago

#46209 new defect (bug)

mediaelement CSS loading at bottom of body

Reported by: antpb Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 4.9
Component: Media Keywords: needs-patch
Focuses: javascript, coding-standards Cc:
PR Number:

Description (last modified by SergeyBiryukov)

The question was asked in #42751 why we are loading mediaelement css in the body. It was agreed in the recent Media meeting that this is better fit for it's own ticket for discussion.


Mentioned by @themezly :

While we are at it, why are we loading mediaelement CSS files at the bottom of body

http://prntscr.com/hh0808 ?

CSS files should always be in head.

Change History (8)

#1 @SergeyBiryukov
10 months ago

  • Description modified (diff)

#2 @joemcgill
8 months ago

  • Milestone changed from Awaiting Review to 5.3

I'm guessing this is related to #44484, and specifically the changes that were introduced in [41877]. We should look at updating both.

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.


3 months ago

#4 @davidbaumwald
3 months ago

This ticket needs a decision and a patch. With Beta 1 of version 5.3 only a few days away, this is being moved to Future Release. If this ticket can be resolved in time for 5.3, feel free to re-milestone.

#5 @mikeschroder
3 months ago

  • Focuses javascript added
  • Keywords needs-patch added
  • Version set to 4.9

Should this be classified as a bug like #44484, mentioned earlier in the ticket, is?

#6 @desrosj
3 months ago

  • Type changed from enhancement to defect (bug)

I'm fine with re-classifying this one.

This ticket was mentioned in Slack in #core-media by mike. View the logs.


2 months ago

#8 @davidbaumwald
2 months ago

  • Milestone changed from 5.3 to Future Release
Note: See TracTickets for help on using tickets.