From 2d18576becda59c73db4bdd61003396e2e75cd4e Mon Sep 17 00:00:00 2001
From: Santhosh Thottingal <santhosh.thottingal@gmail.com>
Date: Tue, 16 Oct 2012 16:08:01 +0530
Subject: [PATCH] Use default font only if webfonts is enabled.

Also contains fixes to make code jshint valid.

Change-Id: I40577acfc939512c71ffe607c14a9fe795760588
---
 resources/js/ext.uls.webfonts.js | 30 ++++++++++++++++++++++--------
 1 file changed, 22 insertions(+), 8 deletions(-)

diff --git a/resources/js/ext.uls.webfonts.js b/resources/js/ext.uls.webfonts.js
index d327aa72..c990958c 100644
--- a/resources/js/ext.uls.webfonts.js
+++ b/resources/js/ext.uls.webfonts.js
@@ -17,11 +17,13 @@
  * @licence MIT License
  */
 ( function ( $, mw, document, undefined ) {
-	"use strict";
+	'use strict';
 
 	$( document ).ready( function () {
-		var mediawikiFontRepository = $.webfonts.repository;
-		var webfontsPreferences = mw.uls.preferences( 'webfonts' );
+		var mediawikiFontRepository, webfontsPreferences;
+
+		mediawikiFontRepository = $.webfonts.repository;
+		webfontsPreferences = mw.uls.preferences( 'webfonts' );
 		mediawikiFontRepository.base = mw.config.get( 'wgExtensionAssetsPath' )
 			+ '/UniversalLanguageSelector/data/fontrepo/fonts/';
 
@@ -34,23 +36,35 @@
 		// Initialize webfonts
 		$( 'body' ).webfonts( {
 			fontSelector: function ( repository, language ) {
-				var font = webfontsPreferences.get( language );
+				var font, enabled;
+
+				font = webfontsPreferences.get( language );
+				enabled = webfontsPreferences.get( 'webfonts-enabled' );
+				// If the user didn't set anything, the preference will be undefined.
+				// The default for now is to enable webfonts if the user didn't select anything.
+				if ( enabled === undefined ) {
+					enabled = true;
+				}
+
 				if ( !font ) {
 					font = repository.defaultFont( language );
 				}
-				if ( font === 'system' ) {
+
+				if ( font === 'system' || !enabled ) {
 					font = null;
 				}
+
 				return font;
 			},
-			exclude: function () {
+			exclude: ( function () {
 				if ( mw.user.options.get( 'editfont' ) ) {
 					// Exclude textboxes from webfonts if user has edit area font option
 					// set using 'Preferences' page
 					return 'textarea';
 				}
+
 				return $.fn.webfonts.defaults.exclude;
-			} ()
+			}() )
 		} );
 	} );
-} ( jQuery, mediaWiki, document ) );
+}( jQuery, mediaWiki, document ) );