Opened 10 years ago

Closed 7 years ago

#9530 closed defect (patchwelcome)

Edit dijit Bug within http://archive.dojotoolkit.org/nightly/dojotoolkit/dojox/grid/tests/test_edit_dijit.html

Reported by: ganeshpuri Owned by:
Priority: high Milestone: future
Component: DojoX Grid Version: 1.3.1
Keywords: edit dijit editor freeze Cc: Douglas Hays
Blocked By: Blocking:

Description

The dijit CurrencyTextBox? Editor in the mentioned testcase freezes when the second row of a column is edited. Here's how to reproduce it:

  1. open http://archive.dojotoolkit.org/nightly/dojotoolkit/dojox/grid/tests/test_edit_dijit.html in IE7
  1. Double click row 1 in the next to last column (column 7, Amount) to edit
  1. Double click row 2 in the next to last column to edit

When the second row is clicked the editor freezes - no cursor, no editing possible. This happens with any two rows: Always the first editing after page load works but all subsequent attempts to edit a cell of column 7 (Amount) freeze the editor.

The bug also occurs in IE8 in IE7 mode. IE8 in IE8 mode doesn't display the grid of this testcase at all.

Attachments (1)

FromJamesDevine.txt (884 bytes) - added by Martin Jakobi 10 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 10 years ago by ganeshpuri

I actually came across the same bug with a NumberTextBox? inside dojox.grid.DataGrid? - shows the same behaviour.

comment:2 Changed 10 years ago by Adam Peller

Cc: Douglas Hays added

Doug, do you have a chance to take a quick look at this?

comment:3 Changed 10 years ago by Douglas Hays

Happens with IE6 as well. I think the Grid's _EditManager is doing some unusual things to handle IE focus that's not working as intended.

Changed 10 years ago by Martin Jakobi

Attachment: FromJamesDevine.txt added

comment:4 Changed 10 years ago by Martin Jakobi

James Devine posted the following exerpt (his description and workaround)

attachment:FromJamesDevine.txt

comment:5 Changed 10 years ago by Betelgeuse

Here's a bit simpler work around that doesn't need creating separate classes for all widget types:

dojox.grid.cells._Base.prototype._finish = function(inRowIndex) {
    this.cancelFormatNode();
}

comment:6 Changed 10 years ago by Nathan Toone

Milestone: 1.41.5
Owner: changed from Bryan Forbes to Nathan Toone

I'll Take this...

comment:7 in reply to:  6 ; Changed 10 years ago by rledousa

Replying to toonetown:

I'll Take this...

Hi - ran into the same problem. Two quick questions -

1) ETA for fix is dojo 1.5? 2) Does trac give me a way to track a bug via email or some other kind of alert so I can watch for changes?

comment:8 in reply to:  7 Changed 10 years ago by Nathan Toone

Replying to rledousa:

1) ETA for fix is dojo 1.5?

Yes - I won't have time to look into this as we have already cut the beta for 1.4

Replying to rledousa:

2) Does trac give me a way to track a bug via email or some other kind of alert so I can watch for changes?

Yes - just add yourself to the "Cc" field (your email address will do) - you can also change your trac account settings.

comment:9 Changed 9 years ago by Nathan Toone

Milestone: 1.5future
Owner: Nathan Toone deleted

Moving my tickets to future, as I am not currently working on them.

comment:10 Changed 7 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:11 Changed 7 years ago by bill

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