diff --git a/src/sql/workbench/contrib/notebook/browser/cellToolbarActions.ts b/src/sql/workbench/contrib/notebook/browser/cellToolbarActions.ts index 8c51634aa8..82373288cf 100644 --- a/src/sql/workbench/contrib/notebook/browser/cellToolbarActions.ts +++ b/src/sql/workbench/contrib/notebook/browser/cellToolbarActions.ts @@ -20,6 +20,7 @@ import { IContextMenuService } from 'vs/platform/contextview/browser/contextView import { StandardKeyboardEvent } from 'vs/base/browser/keyboardEvent'; import { MoveDirection } from 'sql/workbench/services/notebook/browser/models/modelInterfaces'; +const moreActionsLabel = localize('moreActionsLabel', "More"); export class EditCellAction extends ToggleableAction { // Constants @@ -151,10 +152,11 @@ export class CellToggleMoreActions { public onInit(elementRef: HTMLElement, context: CellContext) { this._moreActionsElement = elementRef; + this._moreActionsElement.setAttribute('aria-haspopup', 'menu'); if (this._moreActionsElement.childNodes.length > 0) { this._moreActionsElement.removeChild(this._moreActionsElement.childNodes[0]); } - this._moreActions = new ActionBar(this._moreActionsElement, { orientation: ActionsOrientation.VERTICAL }); + this._moreActions = new ActionBar(this._moreActionsElement, { orientation: ActionsOrientation.VERTICAL, ariaLabel: moreActionsLabel }); this._moreActions.context = { target: this._moreActionsElement }; let validActions = this._actions.filter(a => a instanceof Separator || a instanceof CellActionBase && a.canRun(context)); removeDuplicatedAndStartingSeparators(validActions); @@ -343,7 +345,7 @@ export class CollapseCellAction extends CellActionBase { export class ToggleMoreActions extends Action { private static readonly ID = 'toggleMore'; - private static readonly LABEL = localize('toggleMore', "Toggle More"); + private static readonly LABEL = moreActionsLabel; private static readonly ICON = 'masked-icon more'; constructor(