wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-11-15 10:35:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Fix: focus order and screen reader issue in help popup dialog"

Browse source
This commit is contained in:
jenkins-bot 2020-07-13 19:58:59 +00:00 committed by Gerrit Code Review
parent c17c7047ef 92f2fb1522
commit b097557733
1 changed files with 20 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
modules/ve-mw/ui/tools/ve.ui.MWPopupTool.js
View file

@ -24,6 +24,11 @@ ve.ui.MWPopupTool = function VeUiMWPopupTool( title, toolGroup, config ) {
// Parent constructor
ve.ui.MWPopupTool.super.call( this, toolGroup, config );
this.popup.connect( this, {
ready: 'onPopupOpened',
closing: 'onPopupClosing'
} );
this.$element.addClass( 've-ui-mwPopupTool' );
this.$link.on( 'click', this.onToolLinkClick.bind( this ) );
@ -33,6 +38,20 @@ ve.ui.MWPopupTool = function VeUiMWPopupTool( title, toolGroup, config ) {
OO.inheritClass( ve.ui.MWPopupTool, OO.ui.PopupTool );
/**
* Handle to call when popup is opened.
*/
ve.ui.MWPopupTool.prototype.onPopupOpened = function () {
this.popup.closeButton.focus();
};
/**
* Handle to call when popup is closing
*/
ve.ui.MWPopupTool.prototype.onPopupClosing = function () {
this.$link.trigger( 'focus' );
};
/**
* Handle clicks on the main tool button.
*
@ -193,6 +212,7 @@ ve.ui.MWHelpPopupTool = function VeUiMWHelpPopupTool( toolGroup, config ) {
);
ve.targetLinksToNewWindow( this.$items[ 0 ] );
this.popup.$body.append( this.$items );
this.popup.$element.attr( 'aria-label', ve.msg( 'visualeditor-help-tool' ) );
};
/* Inheritance */