Opened 7 years ago

Closed 6 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 6 years ago by rolweber

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.

comment:2 Changed 6 years ago by bsnelderwaard

So a fix would be to just remove that function createView() from EnhancedGrid?.js again, right? Seems to work for me.

Version 0, edited 6 years ago by bsnelderwaard (next)

comment:3 Changed 6 years ago by craign

Yup, as per the diff in #17375. Works for me too.

comment:4 Changed 6 years ago by bill

Resolution: patchwelcome
Status: newclosed

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.

Note: See TracTickets for help on using tickets.