Opened 13 years ago
Closed 9 years ago
#9464 closed defect (patchwelcome)
vertical scroll position is reset when resizing columns of a grid
Reported by: | Mars Agliullin | Owned by: | |
---|---|---|---|
Priority: | high | Milestone: | future |
Component: | DojoX Grid | Version: | 1.3.0 |
Keywords: | grid vertical scroll | Cc: | |
Blocked By: | Blocking: |
Description
How to reproduce with attached html:
- Make browser window small, so there is vertical scrollbar.
- Scroll it to the bottom.
- Resize any column of the lower grid.
What happens: vertical scrollbar's position is reset to top
What is expected: column resize should not impact scrollbar's position
This behaviour was observed on FF3.0 & IE6. Didn't test on other browsers.
Cause of bug: body of the grid with enabled autoHeight is hidden before column resize. Without grid's body scrollbar may disappear if there's enough space. When grid's body is shown again scrollbar's position is reset to top.
Idea of fix: explicitly replace grid's height with its actual value before resize, so vertical size of content doesn't change. Reset height to 'auto' afterwards.
BTW, the same behaviour is observed when columns are sorted. Attached patch doesn't fix that.
Attachments (2)
Change History (7)
Changed 13 years ago by
Attachment: | pane1_grid.html added |
---|
Changed 13 years ago by
Attachment: | grid_vscroll_patch.diff added |
---|
comment:1 Changed 13 years ago by
Milestone: | tbd → 1.5 |
---|---|
Owner: | changed from Bryan Forbes to Nathan Toone |
comment:2 Changed 13 years ago by
comment:3 Changed 12 years ago by
Milestone: | 1.5 → future |
---|---|
Owner: | Nathan Toone deleted |
Moving my tickets to future, as I am not currently working on them.
comment:4 Changed 9 years ago by
comment:5 Changed 9 years ago by
Resolution: | → patchwelcome |
---|---|
Status: | new → closed |
My patch is under CLA. Just forgot to mark it as such.