From 3b9134f8ac498be8b400c3ce9b2c5a01ac83492a Mon Sep 17 00:00:00 2001 From: Sam Smith Date: Tue, 15 Jun 2021 14:52:06 +0100 Subject: [PATCH] launchULS: Add context to interface.language.change hook Now that the interface.language.change hook is fired when the user switches language using the Compact Language Switcher, we should provide additional context to the UniversalLanguageSelector instrument. Bug: T280770 Change-Id: I6c4ddf9738e527ac42581c9fb4924ed1d12748b4 --- resources/js/ext.uls.launch.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/resources/js/ext.uls.launch.js b/resources/js/ext.uls.launch.js index ac0d5e7c..ebba612d 100644 --- a/resources/js/ext.uls.launch.js +++ b/resources/js/ext.uls.launch.js @@ -61,7 +61,10 @@ function launchULS( $trigger, languagesObject, forCLS ) { return; } - mw.hook( 'mw.uls.interface.language.change' ).fire( language ); + // TODO: The name of this hook should probably be changed to reflect that it covers + // both the user changing their interface language and the user switching to a + // different language. + mw.hook( 'mw.uls.interface.language.change' ).fire( language, 'content-language-switcher' ); location.href = languagesObject[ language ].href; },