Comment and whitespace cleanup
Change-Id: I7ac44f4d371faa3c6941ccf8a4654a035e9b05a5
This commit is contained in:
@@ -156,11 +156,11 @@
|
|||||||
$loginCta.find( 'a' ).click( function ( event ) {
|
$loginCta.find( 'a' ).click( function ( event ) {
|
||||||
event.preventDefault();
|
event.preventDefault();
|
||||||
// Because browsers navigate away when clicking a link,
|
// Because browsers navigate away when clicking a link,
|
||||||
// we are are overriding the normal click behavior to
|
// we are overriding the normal click behavior to allow
|
||||||
// allow the event be logged first - currently there is no
|
// the event be logged first - currently there is no
|
||||||
// local queue for events. Since the hook system does not
|
// local queue for events. Since the hook system does not
|
||||||
// allow returning values, we have this ugly event logging
|
// allow returning values, we have this ugly hack
|
||||||
// specific hack to delay the page load if event logging
|
// for event logging to delay the page loading if event logging
|
||||||
// is enabled. The promise is passed to the hook, so that
|
// is enabled. The promise is passed to the hook, so that
|
||||||
// if event logging is enabled, in can resole the promise
|
// if event logging is enabled, in can resole the promise
|
||||||
// immediately to avoid extra delays.
|
// immediately to avoid extra delays.
|
||||||
@@ -250,8 +250,8 @@
|
|||||||
* Prepare the more languages button. It is a ULS trigger
|
* Prepare the more languages button. It is a ULS trigger
|
||||||
*/
|
*/
|
||||||
prepareMoreLanguages: function () {
|
prepareMoreLanguages: function () {
|
||||||
var displaySettings = this,
|
var $languages, $moreLanguagesButton,
|
||||||
$languages, $moreLanguagesButton;
|
displaySettings = this;
|
||||||
|
|
||||||
$languages = this.$template.find( 'div.uls-ui-languages' );
|
$languages = this.$template.find( 'div.uls-ui-languages' );
|
||||||
$moreLanguagesButton = $( '<button>' )
|
$moreLanguagesButton = $( '<button>' )
|
||||||
@@ -337,6 +337,7 @@
|
|||||||
*/
|
*/
|
||||||
preview: function ( language ) {
|
preview: function ( language ) {
|
||||||
var displaySettings = this;
|
var displaySettings = this;
|
||||||
|
|
||||||
// Reset the language and font for the panel.
|
// Reset the language and font for the panel.
|
||||||
this.$template.attr( 'lang', language )
|
this.$template.attr( 'lang', language )
|
||||||
.css( 'font-family', '' );
|
.css( 'font-family', '' );
|
||||||
@@ -568,6 +569,7 @@
|
|||||||
*/
|
*/
|
||||||
apply: function () {
|
apply: function () {
|
||||||
var displaySettings = this;
|
var displaySettings = this;
|
||||||
|
|
||||||
if ( !displaySettings.dirty ) {
|
if ( !displaySettings.dirty ) {
|
||||||
// No changes to save in this module.
|
// No changes to save in this module.
|
||||||
return;
|
return;
|
||||||
|
|||||||
Reference in New Issue
Block a user