Mooeypoo has uploaded a new change for review.
https://gerrit.wikimedia.org/r/88921
Change subject: Language widget ULS call fix
......................................................................
Language widget ULS call fix
Because of the change to ButtonWidget, the button returned false after
onClick and as a result, ULS dialog didn't open.
This fix adds a fake 'href' value to the PushButtonWidget instance
in ve.ui.LanguageInputWidget so clicking the 'change language' button
continues to evoke ULS.
(Notice, this widget will soon be split into ve-MW with a smaller fallback
widget in core, but for now, this fixes the fact ULS doesn't appear when it
should)
Change-Id: I32eabdc5ee1b3681c20c756f45a3257c7a0b5681
---
M modules/ve/ui/widgets/ve.ui.LanguageInputWidget.js
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor
refs/changes/21/88921/1
diff --git a/modules/ve/ui/widgets/ve.ui.LanguageInputWidget.js
b/modules/ve/ui/widgets/ve.ui.LanguageInputWidget.js
index b84ee63..26c266b 100644
--- a/modules/ve/ui/widgets/ve.ui.LanguageInputWidget.js
+++ b/modules/ve/ui/widgets/ve.ui.LanguageInputWidget.js
@@ -76,6 +76,8 @@
// Create a 'change language' Button:
this.$button = new ve.ui.PushButtonWidget({
'label': ve.msg(
'visualeditor-languageinspector-widget-changelang' ),
+ // Add 'href' so the button returns true on click and triggers
ULS
+ 'href': '#',
'flags': ['primary']
});
--
To view, visit https://gerrit.wikimedia.org/r/88921
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I32eabdc5ee1b3681c20c756f45a3257c7a0b5681
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Mooeypoo <mor...@gmail.com>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
