Skip to content

Commit 17c32e3

Browse files
authored
dxGantt: fix context menu to make it accessible (T1269329) (#28774)
1 parent d3a5bcf commit 17c32e3

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

packages/devextreme/js/ui/gantt/ui.gantt.treelist.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ export class GanttTreeList {
109109
if(e.row?.rowType === 'data') {
110110
this.setOption('selectedRowKeys', [e.row.data[this._gantt.option('tasks.keyExpr')]]);
111111
}
112-
e.items = [];
112+
113113
const info = {
114114
cancel: false,
115115
event: e.event,
@@ -118,6 +118,7 @@ export class GanttTreeList {
118118
position: { x: e.event.pageX, y: e.event.pageY }
119119
};
120120
this._gantt._showPopupMenu(info);
121+
e.event.preventDefault();
121122
}
122123

123124
_getHeight() {

packages/devextreme/testing/tests/DevExpress.ui.widgets/ganttParts/contextMenu.tests.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ QUnit.module('Context Menu', moduleConfig, () => {
4040
assert.equal(getContextMenuElement().length, 0, 'menu is hidden on create');
4141
const $cellElement = $(this.instance._treeList.getCellElement(0, 0));
4242
$cellElement.trigger('contextmenu');
43-
assert.equal(getContextMenuElement().length, 2, 'menu is visible after right click in tree list');
43+
assert.equal(getContextMenuElement().length, 1, 'menu is visible after right click in tree list');
4444
});
4545
test('shown at correct position', function(assert) {
4646
this.createInstance(options.allSourcesOptions);

0 commit comments

Comments
 (0)