Opened 10 years ago

Closed 10 years ago

#9095 closed defect (invalid)

grid's height becomes extreme high when it has only one row

Reported by: coolhair88 Owned by: Bryan Forbes
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.3.0
Keywords: grid Cc:
Blocked By: Blocking:

Description

when the "font-family" is set to "Verdana", and the "font-size" is set to "13px" - "15px", and "autoHeight" is ture, and the grid just has only one row, its height becomes extreme high.

I tested it in several computers and found that:

when the "font-size" set to "15px", it happens on Firefox

when the "font-size" set to "14px", it happens on IE6, IE7 and Google Chrome

My test code is here: http://pastie.org/440374

Attachments (1)

z.html (2.1 KB) - added by coolhair88 10 years ago.
this is a test case, open it with different browsers

Download all attachments as: .zip

Change History (5)

comment:1 Changed 10 years ago by coolhair88

I finally find that problem is casued by measurePage function in scroller.js

the last line of the function is "return (this.rowsPerPage == h)?(h*this._defaultRowHeight):h;"

and it returns true when grid just has one row sometimes

comment:2 Changed 10 years ago by bill

Milestone: 1.4tbd
Priority: highestnormal

BTW you can (and should) attach test cases to the ticket (after submitting the original ticket) by using the "attach file" button. That paste you had is missing at least the dojo.require() statements.

You should also let the grid owner mark a milestone, since it means that they are committing (or at least hoping) to have the fix by a certain release.

But having said that, thanks for coming up with a test case and tracking down the problem.

Changed 10 years ago by coolhair88

Attachment: z.html added

this is a test case, open it with different browsers

comment:3 in reply to:  2 Changed 10 years ago by coolhair88

Replying to bill:

BTW you can (and should) attach test cases to the ticket (after submitting the original ticket) by using the "attach file" button. That paste you had is missing at least the dojo.require() statements.

You should also let the grid owner mark a milestone, since it means that they are committing (or at least hoping) to have the fix by a certain release.

But having said that, thanks for coming up with a test case and tracking down the problem.

I'm sorry. I'm new here and there are many things that I didn't know. I think it's a serious problem and should be fixed as quickly as possible.

comment:4 Changed 10 years ago by Nathan Toone

Resolution: invalid
Status: newclosed

This is the compat grid - which has not been supported for quite some time. Please resubmit a test case using the current grid (dojox.grid.DataGrid?) - and reopen the bug if you are still seeing the issue.

Note: See TracTickets for help on using tickets.