From d217a2fe73d57d9689df750c4a8771aaed179a37 Mon Sep 17 00:00:00 2001 From: Santhosh Thottingal Date: Fri, 5 Jul 2013 15:15:56 +0530 Subject: [PATCH] Make anything with class 'uls-settings-trigger' work like ULS trigger Usecase: Reading problems? click [here]. That [here] can be a ULS trigger to open language settings. Change-Id: I1561f8b782037bfdb418f33097fc8b4c519ef393 --- resources/js/ext.uls.interface.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/resources/js/ext.uls.interface.js b/resources/js/ext.uls.interface.js index 02aca379..7f19f789 100644 --- a/resources/js/ext.uls.interface.js +++ b/resources/js/ext.uls.interface.js @@ -243,7 +243,8 @@ .attr( 'title', $.i18n( 'ext-uls-language-settings-title' ) ); // Append ULS cog to languages section, but make sure it is visible. $pLang.show().prepend( $ulsSettingsTrigger ); - + // Take care of any other elements with this class. + $ulsSettingsTrigger = $( '.uls-settings-trigger' ); // Remove the dummy link, which was added to make sure that the section appears $pLang.find( '.uls-p-lang-dummy' ).remove(); @@ -289,8 +290,8 @@ var topRowHeight, caretHeight, caretWidth, $caretBefore = $( '' ).addClass( 'caret-before' ), $caretAfter = $( '' ).addClass( 'caret-after' ), - ulsTriggerWidth = $ulsSettingsTrigger.width(), - ulsTriggerOffset = $ulsSettingsTrigger.offset(); + ulsTriggerWidth = this.$element.width(), + ulsTriggerOffset = this.$element.offset(); // Add the callout caret triangle // pointing to the trigger icon