mirror of
https://gerrit.wikimedia.org/r/mediawiki/skins/MinervaNeue
synced 2024-12-12 14:25:15 +00:00
2b41416c0d
Currently `mw.util.addPortletLink` cannot properly add a portlet link to the associated pages tabs, as there is no `p-associated-pages` id on mobile. This change pulls the id from the page data, and adds the necessary class for the tab to be styled correctly - since tabs do not have corresponding icons while most portlet links do, we also branch on this class (effectively on whether we are in the tab container) to ensure an icon is not inserted Finally, I added a few comments and spacing in the sections of code that I touched to make them more readable and resolve some of the linter warnings, but happy to hear if these are not helpful! Bug: T340728 Change-Id: I33fc12611a6238552a3eb47f6ca37f087903a92a |
||
|---|---|---|
| .. | ||
| page-issues | ||
| AB.js | ||
| addPortletLink.js | ||
| ctaDrawers.js | ||
| downloadPageAction.js | ||
| drawers.js | ||
| initMobile.js | ||
| menu.js | ||
| mobileRedirect.js | ||
| preInit.js | ||
| references.js | ||
| search.js | ||
| searchSuggestReveal.js | ||
| setup.js | ||
| styles.less | ||
| TabScroll.js | ||
| TitleUtil.js | ||
| toc.less | ||
| Toolbar.js | ||
| Toolbar.less | ||
| UriUtil.js | ||
| watchstar.js | ||
| watchStar.less | ||