Caused by https://gerrit.wikimedia.org/r/#/c/224966/15/src/widgets/PopupWidget.js
Also impacts StructuredDiscussions's mention inspector.
Caused by https://gerrit.wikimedia.org/r/#/c/224966/15/src/widgets/PopupWidget.js
Also impacts StructuredDiscussions's mention inspector.
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
PopupWidget: Remove 'focusout' handling again, limit to CapsuleMultiSelectWidget | oojs/ui | master | +24 -24 |
Looks like this line:
!OO.ui.contains( widget.$element, document.activeElement, true ) &&
Should be like this:
!OO.ui.contains( widget.$element[0], document.activeElement, true ) &&
No promises that this is the only issue, but VE's inspectors seem to work correctly with that patch.
Hmm, actually, it gets more interesting for LanguageInspector, which can open a dialog. Opening the dialog shouldn't close the inspector, but it does. I think it would be safer to remove the general 'focusout' handler from PopupWidget.
Change 231338 had a related patch set uploaded (by Bartosz Dziewoński):
PopupWidget: Remove 'focusout' handling again, limit to CapsuleMultiSelectWidget
Change 231338 merged by jenkins-bot:
PopupWidget: Remove 'focusout' handling again, limit to CapsuleMultiSelectWidget