mirror of
https://gerrit.wikimedia.org/r/mediawiki/skins/Vector.git
synced 2024-12-04 20:18:57 +00:00
9e4bdd00bf
Rather than try to build individually build login, logout, and create account menu items, we handle them the same as all other user links menu items in Hooks.php. While Hooks isnt ideal, there currently isnt a good path for moving that code to SkinComponents until core provides menu data. In the meantime, this patch reduces code complexity and prevents bugs like T324638. This approach also allows us to move user links logic from SkinVector22 to VectorComponentUserLinks.php, and ensures the user links dropdown contains multiple menus, which allows us to reuse styles from page tools dropdowns. Expected 11 visual changes: * minor visual change where the user links dropdown has an additional 4px vertical padding. This padding was originally added to the page tools dropdowns per Alex's request, but Alex also said that dropdowns should share the same spacing. This change makes the styles all consistent * Order of talk and contribution links have been swapped Bug: T289212 Bug: T319356 Bug: T328954 Change-Id: Iac0586893fec26a8a6c2c904ce08fbf1e19b339c |
||
---|---|---|
.. | ||
__mocks__ | ||
__snapshots__ | ||
skins.vector.es6 | ||
skins.vector.js | ||
AB.test.js | ||
App.test.js | ||
deferUntilFrame.test.js | ||
dropdownMenu.test.js | ||
fetch.test.js | ||
instrumentation.test.js | ||
pinnableElement.test.js | ||
restSearchClient.test.js | ||
stickyHeader.test.js | ||
tableOfContents.test.js | ||
urlGenerator.test.js | ||
userLinks.test.js | ||
userLinksData.js |