From 3dda6b58286b868d419a089c8bc6871c83319639 Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Fri, 22 Nov 2013 10:18:20 +0530 Subject: [PATCH] Comment and whitespace cleanup Change-Id: I7ac44f4d371faa3c6941ccf8a4654a035e9b05a5 --- resources/js/ext.uls.displaysettings.js | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/resources/js/ext.uls.displaysettings.js b/resources/js/ext.uls.displaysettings.js index 07f15299..db70e419 100644 --- a/resources/js/ext.uls.displaysettings.js +++ b/resources/js/ext.uls.displaysettings.js @@ -156,11 +156,11 @@ $loginCta.find( 'a' ).click( function ( event ) { event.preventDefault(); // Because browsers navigate away when clicking a link, - // we are are overriding the normal click behavior to - // allow the event be logged first - currently there is no + // we are overriding the normal click behavior to allow + // the event be logged first - currently there is no // local queue for events. Since the hook system does not - // allow returning values, we have this ugly event logging - // specific hack to delay the page load if event logging + // allow returning values, we have this ugly hack + // for event logging to delay the page loading if event logging // is enabled. The promise is passed to the hook, so that // if event logging is enabled, in can resole the promise // immediately to avoid extra delays. @@ -250,8 +250,8 @@ * Prepare the more languages button. It is a ULS trigger */ prepareMoreLanguages: function () { - var displaySettings = this, - $languages, $moreLanguagesButton; + var $languages, $moreLanguagesButton, + displaySettings = this; $languages = this.$template.find( 'div.uls-ui-languages' ); $moreLanguagesButton = $( '