Despite skins.minervas.icons.images and ext.gather.menu.icon both specifying 'position' => 'bottom' in RL definition it seems to be ignored and these get loaded in the head.
Description
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Duplicate | • Jhernandez | T104432 [EPIC]: Improve mobile site performance | |||
| Resolved | Jdlrobson | T98986 [EPIC] Improve the status quo of mobile web performance | |||
| Duplicate | None | T98397 Main menu and notification css should be loaded via JavaScript | |||
| Duplicate | None | T98987 Epic: Make mobile web more performant | |||
| Resolved | Jdlrobson | T97289 Trim down CSS loaded in the head for anonymous users on mobile | |||
| Invalid | matmarex | T98394 ResourceLoaderImageModule always gets loaded in head |
Event Timeline
Comment Actions
This is the expected behavior, because you load these modules with addModuleStyles. If you loaded them with addModules, they would be at the bottom (and they'd only load on browsers with JavaScript enabled). @Gilles is apparently doing some work to make this not so (T97410?), but I wasn't following it.
Comment Actions
(ResourceLoaderImageModule, in fact, ignores the 'position' option entirely and always uses 'bottom'.)
Comment Actions
Hmm.. Why am I seeing this module load in a style tag in the top of the page? Am I missing a commit in core?
Comment Actions
Just saw your comment.. Doesn't add modules force them to load via JavaScript though?
Comment Actions
Mm.. Of course we can't inline styles at bottom and ensure cache validation can we... :-/