From 8e85f0d611a3ac35e340e0920d9969c9895de152 Mon Sep 17 00:00:00 2001 From: Sam Smith Date: Tue, 11 May 2021 12:15:55 +0100 Subject: [PATCH] Fire hook when compact language links are opened Following on from Icc64044d, we fire the mw.uls.compact_language_links.open hook whenever that treatment is opened (and not only the first time that the triggering element is clicked). Bug: T281928 Change-Id: I6fa966c22c738ca12bcbeb0225c03f3f8394ecee --- resources/js/ext.uls.interface.js | 2 -- resources/js/ext.uls.launch.js | 2 ++ 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/resources/js/ext.uls.interface.js b/resources/js/ext.uls.interface.js index 497133d4..1518afee 100644 --- a/resources/js/ext.uls.interface.js +++ b/resources/js/ext.uls.interface.js @@ -473,8 +473,6 @@ !standalone ); - mw.hook( 'mw.uls.compact_language_links.open' ).fire(); - // Trigger the click handler to open ULS once ready if ( standalone ) { // Provide access to display and input settings if this entry point is the single point diff --git a/resources/js/ext.uls.launch.js b/resources/js/ext.uls.launch.js index e2bfe9de..766a28ed 100644 --- a/resources/js/ext.uls.launch.js +++ b/resources/js/ext.uls.launch.js @@ -91,6 +91,8 @@ function launchULS( $trigger, languagesObject, forCLS ) { }, onVisible: function () { $trigger.addClass( 'selector-open' ); + + mw.hook( 'mw.uls.compact_language_links.open' ).fire(); }, languageDecorator: function ( $languageLink, language ) { var element = languagesObject[ language ];