Opened 8 years ago
Closed 7 years ago
#17130 closed defect (patchwelcome)
dojox.grid.EnhancedGrid column rearrangement broken in Firefox
Reported by: | jeffreym | Owned by: | Evan |
---|---|---|---|
Priority: | undecided | Milestone: | tbd |
Component: | DojoX Grid | Version: | 1.9.0 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Dragging an EnhancedGrid? column header to move it doesn't work in Firefox (unless you drag from near the top or bottom of the header cell); it resizes the column instead. This appears to be due to code in EnhancedGrid?.createView which was intended to work around _HeaderBuilder.getCellX giving incorrect values (#12115; r23433). That bug was fixed as a side effect of a rewrite of the function (#16924; r31022), so now EnhancedGrid?'s workaround is giving incorrect values in the other direction.
Change History (4)
comment:1 Changed 8 years ago by
comment:2 Changed 7 years ago by
So a fix would be to just remove that function createView() from EnhancedGrid?.js again, right? Seems to work for me.
comment:4 Changed 7 years ago by
Resolution: | → patchwelcome |
---|---|
Status: | new → closed |
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.
This also breaks sorting in an EnhancedGrid?. When you try to click on a header to change the sort order, that is also processed as column resize. Some columns on the left might still work partially. But further to the right, everything's broken.