The main menu is currently sent down the wire as HTML but only shown with JS. Let's change this.
This should reduce overall page HTML and reduce code duplication. Welcome to the future!
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Generate menu HTML via JavaScript. | mediawiki/extensions/MobileFrontend | master | +71 -42 |
| 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 | |||
| Resolved | phuedx | T94458 Mobile menu improvements (alpha) | |||
| Resolved | • bmansurov | T96172 On alpha, clicking header launches hamburger menu and search without forcing a page load | |||
| Resolved | • bmansurov | T95332 Stop serving menu and styles in the page HTML | |||
| Resolved | Jdlrobson | T97431 Render main menu via a template |
According to the page-ui-daily graph, search is clicked ~5 times more than the hamburger. Let's definitely render the menu when it's requested!
Exactly. It sounds like the design for the main menu is in flux but moving to templates will make this easier to manage in future.
We'll see. I certainly think it'll make it easier to change the menu design in the future.
This task can be resolved by working on T96172 and promoting the new design to beta/stable.
Change 207844 had a related patch set uploaded (by Jdlrobson):
Generate menu HTML via JavaScript.