Cleanup

whitepace, unwanted fixmes, wrong comments, jshint

Change-Id: I571b2b3a7e9fbbe7c93262ee9783fbe334d9170e

resources/js/ext.uls.displaysettings.js

@@ -84,7 +84,7 @@
 		+ '<button id="uls-displaysettings-apply" class="active blue button" data-i18n="ext-uls-language-settings-apply"></button>'
 		+ '</div>'
 		+ '</div>'
-		+ '</div>'; // FIXME i18n and too much hardcoding.
+		+ '</div>';
 
 	function DisplaySettings( $parent ) {
 		this.name = $.i18n( 'ext-uls-display-settings-title-short' );

resources/js/ext.uls.ime.js

@@ -1,5 +1,5 @@
 /**
- * ULS-Webfonts integration
+ * ULS - jQuery IME integration
  *
  * Copyright (C) 2012 Alolita Sharma, Amir Aharoni, Arun Ganesh, Brandon Harris,
  * Niklas Laxström, Pau Giner, Santhosh Thottingal, Siebrand Mazeland and other
@@ -40,7 +40,7 @@
 		return availableLanguages;
 	};
 
-	mw.ime.getIMELanguageList = function  () {
+	mw.ime.getIMELanguageList = function () {
 		var unique = [],
 			imeLanguageList,
 			previousIMELanguages;
@@ -103,7 +103,7 @@
 			}
 		} );
 
-		 $.ime.preferences.load();
+		$.ime.preferences.load();
 	} );
 
 

resources/js/ext.uls.inputsettings.js

@@ -129,11 +129,9 @@
 				'class': 'imelabel'
 			} );
 
-			// TODO Niklas: Was there some problems with IE needing the type=foo immediately?
-			$inputMethodItem = $( '<input>' ).attr( {
+			$inputMethodItem = $( '<input type="radio">' ).attr( {
 				name: 'ime',
 				id: imeId,
-				type: 'radio',
 				value: imeId,
 				checked: selected
 			} );
@@ -322,7 +320,7 @@
 				that.close();
 			} );
 
-			$imeListContainer.on( 'change', 'input:radio[name=ime]:checked',  function () {
+			$imeListContainer.on( 'change', 'input:radio[name=ime]:checked', function () {
 				var ime = $( this ).val();
 
 				$.ime.preferences.setLanguage( that.imeLanguage );