#14479 closed defect (duplicate)
EnhancedGrid and RangeError
| Reported by: | matej | Owned by: | Evan |
|---|---|---|---|
| Priority: | high | Milestone: | 1.6.2 |
| Component: | DojoX Grid | Version: | 1.7.1 |
| Keywords: | EnhancedGrid, BorderContainer, ContentPane | Cc: | |
| Blocked By: | Blocking: |
Description (last modified by )
When I use EnhancedGrid inside BorderContainer and ContentPane debugger gets a stack overflow.
Error messages:
RangeError: Maximum call stack size exceeded RangeError Deferred.js:216
Attachments (3)
Change History (13)
Changed 10 years ago by
comment:1 Changed 10 years ago by
| Component: | Dijit → DojoX Grid |
|---|---|
| Description: | modified (diff) |
| Owner: | set to Evan |
| Priority: | highest → normal |
comment:3 Changed 10 years ago by
So this was caused by an inappropriate usage - Grid needs an explicit width/height otherwise it can't layout itself correctly.
Also there seems to be a missed width on the left content pane in re.html. Adding a refined version of re.html that works well for me now
comment:4 Changed 10 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
Closing since it's due to an inappropriate usage/test case.
comment:5 Changed 10 years ago by
Well my code worked just fine on 1.6.1.
At the very least this is a regression and should not cause infinite loop.
comment:6 follow-ups: 7 9 Changed 10 years ago by
Evan - why do you need to explicitly set a size on the grid? It should be getting it's size set from the parent ContentPane, which is getting it's size set from the parent BorderContainer.
(It looks like the test case is missing setting padding:0, margin: 0 on body and html, not sure if that's related to anything.)
comment:7 Changed 10 years ago by
| Milestone: | 1.8 → 1.6.2 |
|---|---|
| Resolution: | invalid → duplicate |
Thanks Bill/myersjj,
I finally figured it out, this is a defect and is duplicate with [14504]:
"It's actually a regression caused by [27395], already fixed with [27440](which has been applied to 1.6/1.7/trunk)"
Changed 10 years ago by
| Attachment: | re_evan.html added |
|---|
Adding a newer version - working well with the only change of referring from correct EDG css - <LINK href="dojox/grid/enhanced/resources/tundra/EnhancedGrid.css" rel="stylesheet">
comment:8 follow-up: 10 Changed 10 years ago by
I can confirm that copying the updated Pagination.js file (from #27440) into my 1.7.1 installation resolved the problem. Thanks!
comment:9 Changed 10 years ago by
Replying to bill:
Evan - why do you need to explicitly set a size on the grid? It should be getting it's size set from the parent ContentPane, which is getting it's size set from the parent BorderContainer.
Yes, Bill,
No need to set grid size in this case, just uploaded a newer version
Matej - Again, your test file is referencing a sample.js that doesn't exist. Please attach self contained test files to future tickets.
Evan - this has infinite recursion in the Grid code.