Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

With DnD Extension and dijit/Menu, IE10/11 click on a dgrid does not dismiss menu #1120

Open
frankf2 opened this issue Apr 20, 2015 · 1 comment

Comments

@frankf2
Copy link

frankf2 commented Apr 20, 2015

dojo 1.10.4, dgrid 0.4

A dijit menu displayed in/over a dgrid is not dismissed when that or another dgrid's row is mouse clicked and the clicked row is in a dgrid that uses the DnD Extension. FF, Chrome, and Safari do dismiss the dijit menu.

This was reproduced by modifying dgrid/test/OnDemand.html.

Modify define (and function params: ", Menu, MenuItem, DnD") to include:

"dijit/Menu",
"dijit/MenuItem",
"dgrid/extensions/DnD",

Modify the StandardGrid (dgrid) to include the DnD Extension (this affects all dgrids on the page):

var StandardGrid = declare([Grid, Selection, Keyboard, Editor, DnD]);

Add the dijit menu to the top dgrid:

var menu = new Menu({targetNodeIds: [grid.domNode]});
menu.onBlur = function(e) {
                console.log("menu blur");
};
menu.addChild(new MenuItem({label: "One"}));
menu.addChild(new MenuItem({label: "Two"}));
menu.startup();

Operation:

  • Right click on the top dgrid to display the menu.
  • Click on a row of that or another dgrid (all are using the DnD Extension).
  • In IE10/11, the menu is not dismissed, but is dismissed in other browsers.

Note: Updated for Clarification. The menu will not be dismissed when the click is on a row of a dgrid that uses the DnD Extension. The assumption is that the dgrid with DnD Extension is consuming the focus/blur events. Clicking on a row in a dgrid that does not use the DnD Extension does dismiss the menu.

@kruthikavenkatathri
Copy link

My guess is #1067 is related to this issue. Fixing the correct focus on mousedown should fix all these issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants