Opened 9 years ago

Closed 9 years ago

#16007 closed defect (patchwelcome)

dojo 1.7 dijit.Menu throws a "Can't move focus to the control because it is invisible" error in IE8

Reported by: li_bo701 Owned by: Evan
Priority: undecided Milestone: tbd
Component: DojoX Grid Version: 1.7.2
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

In Dojo 1.7, dijit.Menu will throw an error as below in IE8, Message: Can't move focus to the control because it is invisible, not enabled, or of a type that does not accept the focus.

Line: 12109
Char: 5
Code: 0
URI: http://localhost:9080/zosmf/IzuUICommon/1_7/CustomBuild/IzugDojoCommon.js.uncompressed.js

Reproduce steps,

  1. Add menu to grid
  2. When navigate to this grid at the first time, using right mouse click this grid, the menu will show out, click one of them then this exception will be thrown in IE8, and it won't occur anymore when you right click again.

This issue won't occur in Dojo1.6.

This is the Dojo code causes this problem, in "savedFocusNode.focus()", it seems savedFocusNode is still invisible when you navigate this page at the first time

function closeAndRestoreFocus(){
// user has clicked on a menu or popup
    if(self.refocus && savedFocusNode){

Attachments (1)

defect.png (21.6 KB) - added by li_bo701 9 years ago.

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by li_bo701

Attachment: defect.png added

comment:1 Changed 9 years ago by bill

Component: GeneralDojoX Grid
Description: modified (diff)
Owner: set to Evan

comment:2 Changed 9 years ago by bill

DojoX Grid and EnhancedGrid are deprecated in favor of dgrid and gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.

comment:3 Changed 9 years ago by bill

Resolution: patchwelcome
Status: newclosed
Note: See TracTickets for help on using tickets.