Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#8575 closed defect (fixed)

PATCH (CCLA) Adjusting Cell width while having a cell with a dijit.Editor open causes errors.

Reported by: Jared Jurkiewicz Owned by: Bryan Forbes
Priority: high Milestone: 1.3
Component: DojoX Grid Version: 1.2.3
Keywords: Cc:
Blocked By: Blocking:

Description

Adjusting Cell width while having a cell with a dijit.Editor open causes errors.

This problem can be seen in both the tests/test_edit_dijit.html and in compat/tests/test_edit_dijit.html.

Both problems stem from the same issue, the Editor loads asynchronously and the close/restore function the grid does assumes synchronous. Basically, grid calls editor.setValue() too early and blows up because the editor has not finished initializing.

The good news is, there is a relatively trivial change in the Grid editors helper setValue function. I'll attach the patch shortly. The basic fix is to look for the onLoadDeferred and if present assign a callback to reapply the value of the cell after the grid finishes creating and initializing.

Attachments (1)

RichTextEditor_Grid_20090204.patch (1.3 KB) - added by Jared Jurkiewicz 10 years ago.

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by Jared Jurkiewicz

comment:1 Changed 10 years ago by Bryan Forbes

Resolution: fixed
Status: newclosed

(In [16564]) * Applied patch to fix column resizing while dijit.Editor open causing errors (fixes #8575 !strict).

comment:2 Changed 10 years ago by Adam Peller

Milestone: tbd1.3

batch move of tickets marked 'tbd' fixed in the 1.3 timeframe

Note: See TracTickets for help on using tickets.