Show the web fonts selectors only if they are enabled

Change-Id: I70b9f9468154b542a80a9717afdb414530bd0d37
This commit is contained in:
Amir E. Aharoni
2013-04-23 18:54:11 +03:00
parent 865ac4d380
commit b27d0a902b

View File

@@ -56,17 +56,7 @@
+ '</div>' + '</div>'
+ '</div>' + '</div>'
// Webfonts enabling chechbox with label + '<div id="uls-display-settings-font-selectors">'
+ '<div class="row">'
+ '<div class="eleven columns">'
+ '<label class="checkbox">'
+ '<input type="checkbox" id="webfonts-enable-checkbox" />'
+ '<strong data-i18n="ext-uls-webfonts-settings-title"></strong> '
+ '<span data-i18n="ext-uls-webfonts-settings-info"></span> '
+ '<a target="_blank" href="//www.mediawiki.org/wiki/Special:MyLanguage/Help:Extension:WebFonts" data-i18n="ext-uls-webfonts-settings-info-link"></a>'
+ '</label>'
+ '</div>'
+ '</div>'
// Menus font selection dropdown with label // Menus font selection dropdown with label
+ '<div class="row uls-content-fonts">' + '<div class="row uls-content-fonts">'
@@ -84,6 +74,20 @@
+ '<select id="ui-font-selector" class="four columns end uls-font-select"></select>' + '<select id="ui-font-selector" class="four columns end uls-font-select"></select>'
+ '</div>' + '</div>'
+ '</div>' // End font selectors
// Webfonts enabling chechbox with label
+ '<div class="row">'
+ '<div class="eleven columns">'
+ '<label class="checkbox">'
+ '<input type="checkbox" id="webfonts-enable-checkbox" />'
+ '<strong data-i18n="ext-uls-webfonts-settings-title"></strong> '
+ '<span data-i18n="ext-uls-webfonts-settings-info"></span> '
+ '<a target="_blank" href="//www.mediawiki.org/wiki/Special:MyLanguage/Help:Extension:WebFonts" data-i18n="ext-uls-webfonts-settings-info-link"></a>'
+ '</label>'
+ '</div>'
+ '</div>'
+ '</div>' // End font settings section + '</div>' // End font settings section
// Separator // Separator
@@ -128,7 +132,15 @@
}, },
prepareWebfontsCheckbox: function () { prepareWebfontsCheckbox: function () {
$( '#webfonts-enable-checkbox' ).prop( 'checked', this.isWebFontsEnabled() ); var webFontsEnabled = this.isWebFontsEnabled();
if ( !webFontsEnabled ) {
this.$template.find(
'#uls-display-settings-font-selectors'
).addClass( 'hide' );
}
$( '#webfonts-enable-checkbox' ).prop( 'checked', webFontsEnabled );
}, },
isWebFontsEnabled: function () { isWebFontsEnabled: function () {
@@ -432,9 +444,14 @@
} ); } );
displaySettings.$template.find( '#webfonts-enable-checkbox' ).on( 'click', function () { displaySettings.$template.find( '#webfonts-enable-checkbox' ).on( 'click', function () {
var $fontSelectors = displaySettings.$template.find(
'#uls-display-settings-font-selectors'
);
displaySettings.enableApplyButton(); displaySettings.enableApplyButton();
if ( this.checked ) { if ( this.checked ) {
$fontSelectors.removeClass( 'hide' );
mw.webfonts.preferences.enable(); mw.webfonts.preferences.enable();
mw.webfonts.setup(); mw.webfonts.setup();
displaySettings.$webfonts = $( 'body' ).data( 'webfonts' ); displaySettings.$webfonts = $( 'body' ).data( 'webfonts' );
@@ -446,6 +463,7 @@
displaySettings.$webfonts.apply( $uiFontSelector.find( 'option:selected' ) ); displaySettings.$webfonts.apply( $uiFontSelector.find( 'option:selected' ) );
displaySettings.$webfonts.refresh(); displaySettings.$webfonts.refresh();
} else { } else {
$fontSelectors.addClass( 'hide' );
mw.webfonts.preferences.disable(); mw.webfonts.preferences.disable();
mw.webfonts.preferences.setFont( displaySettings.uiLanguage, 'system' ); mw.webfonts.preferences.setFont( displaySettings.uiLanguage, 'system' );
displaySettings.$webfonts.refresh(); displaySettings.$webfonts.refresh();