Changeset 12742

Timestamp:

02/27/08 08:45:07 (9 months ago)

Author:

jaredj

Message:

Update to fix double-click cell editor close and open problem. Fix provided by Ben Shell. fixes #5842 \!strict

Location:

dojox/trunk/grid

Files:

• VirtualGrid.js (modified) (2 diffs)
• _grid/publicEvents.js (modified) (2 diffs)

dojox/trunk/grid/VirtualGrid.js

@@ -110 +110 @@
         //              Single-click starts editing. Default is double-click
         singleClickEdit: false,
+
+        // Used to store the last two clicks, to ensure double-clicking occurs based on the intended row
+        _click: null,
         
         // private
@@ -138 +141 @@
                 this.styleChanged = this._styleChanged;
                 this.setStructure(this.structure);
+                this._click = [];
         },
         

dojox/trunk/grid/_grid/publicEvents.js

@@ -175 +175 @@
                 // e: Event
                 //              Decorated event object which contains reference to grid, cell, and rowIndex
+                this._click[0] = this._click[1];
+                this._click[1] = e;
                 if(!this.edit.isEditCell(e.rowIndex, e.cellIndex)){
                         this.focus.setFocusCell(e.cell, e.rowIndex);
@@ -186 +188 @@
                 // e: Event
                 //              Decorated event object contains reference to grid, cell, and rowIndex
-                this.edit.setEditCell(e.cell, e.rowIndex); 
+                if(dojo.isIE){
+                        this.edit.setEditCell(this._click[1].cell, this._click[1].rowIndex);
+                }else if(this._click[0].rowIndex != this._click[1].rowIndex){
+                        this.edit.setEditCell(this._click[0].cell, this._click[0].rowIndex);
+                }else{
+                        this.edit.setEditCell(e.cell, e.rowIndex);
+                }
                 this.onRowDblClick(e);
         },