mirror of
https://gerrit.wikimedia.org/r/mediawiki/skins/MinervaNeue
synced 2024-11-23 22:03:39 +00:00
Merge "Revert "The new class should be present alongside the old class for all page views""
This commit is contained in:
commit
978702f3be
|
@ -1,20 +0,0 @@
|
|||
const LEGACY_NIGHT = 'skin-night-mode-clientpref-1';
|
||||
const LEGACY_AUTO = 'skin-night-mode-clientpref-2';
|
||||
const THEME_NIGHT = 'skin-theme-clientpref-night';
|
||||
const THEME_AUTO = 'skin-theme-clientpref-os';
|
||||
|
||||
/**
|
||||
* Prepares for rename in T359983. Ensures that the new classes are present in the DOM on page load.
|
||||
* Since the controls for mobile are only on Special:MobileOptions we only worry about page load and
|
||||
* assume they won't change later on.
|
||||
*/
|
||||
module.exports = function nightModeClasses() {
|
||||
const node = document.documentElement;
|
||||
|
||||
if ( node.classList.contains( LEGACY_AUTO ) ) {
|
||||
node.classList.add( THEME_AUTO );
|
||||
}
|
||||
if ( node.classList.contains( LEGACY_NIGHT ) ) {
|
||||
node.classList.add( THEME_NIGHT );
|
||||
}
|
||||
};
|
|
@ -3,7 +3,6 @@
|
|||
* It should run without errors even if MobileFrontend is not installed.
|
||||
*/
|
||||
var ms = require( 'mobile.startup' ),
|
||||
nightModeClasses = require( './nightModeClasses.js' ),
|
||||
reportIfNightModeWasDisabledOnPage = require( './reportIfNightModeWasDisabledOnPage.js' ),
|
||||
addPortletLink = require( './addPortletLink.js' ),
|
||||
teleportTarget = require( 'mediawiki.page.ready' ).teleportTarget;
|
||||
|
@ -43,7 +42,6 @@ function init() {
|
|||
reportIfNightModeWasDisabledOnPage(
|
||||
document.documentElement, mw.user.options, mw.user.isNamed()
|
||||
);
|
||||
nightModeClasses();
|
||||
}
|
||||
|
||||
init();
|
||||
|
|
|
@ -519,7 +519,6 @@
|
|||
},
|
||||
"packageFiles": [
|
||||
"resources/skins.minerva.scripts/setup.js",
|
||||
"resources/skins.minerva.scripts/nightModeClasses.js",
|
||||
"resources/skins.minerva.scripts/reportIfNightModeWasDisabledOnPage.js",
|
||||
"resources/skins.minerva.scripts/addPortletLink.js",
|
||||
"resources/skins.minerva.scripts/initMobile.js",
|
||||
|
|
Loading…
Reference in a new issue