Opened 9 years ago

Closed 7 years ago

#13003 closed defect (patchwelcome)

erroneous horizontal scroll bar

Reported by: Kate Glickman Owned by: Evan
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.6.1
Keywords: Cc:
Blocked By: Blocking:

Description

I have a grid with 3 columns and 2 lines per row that has enough data that it should scroll. 2 of the columns have fixed widths and one has width:100%.

When the total rowCount is one particular number, the grid seems to know ahead of time that the vertical scroll is needed and leaves space for it to the right of the content. (Although it is still adding a horizontal scroll bar for about one pixel for some reason.) When the total rowCount is a different particular number, it seems to not know that a vertical scroll bar is needed so the scroll bar gets painted right on top of the rightmost content of the grid.

At first I thought the problem was that one of my columns is width: 100% and that it couldn't calculate how many rows were going to be visible ahead of time. However, I changed the grid to specify a rowHeight and the problem still occurs. With a rowHeight, I would have thought it could definitively tell if a vertical scrollbar was necessary.

I am attaching a sample that reproduces the problem. Use rows=11 to see the problem that I am talking about. Use rows=12 to see it more or less working but with an unnecessary horizontal scrollbar with about 1 pixel of empty space.

var rows = 11; can repro vertical scrollbar prob with this var rows = 12; vs works fine but still extraneous hs

Attachments (1)

test_data_grid.html (2.9 KB) - added by Kate Glickman 9 years ago.

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by Kate Glickman

Attachment: test_data_grid.html added

comment:1 Changed 9 years ago by Kate Glickman

Firefox 3.6.17 Windows XP SP3

comment:2 Changed 9 years ago by Kate Glickman

Same problem on IE 8 and Safari 5.0.5

comment:3 Changed 8 years ago by Evan

Owner: changed from evan to Evan

comment:4 Changed 7 years ago by bill

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.

comment:5 Changed 7 years ago by bill

Resolution: patchwelcome
Status: newclosed
Note: See TracTickets for help on using tickets.