#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 9 years ago by
comment:1 Changed 9 years ago by
Component: | Dijit → DojoX Grid |
---|---|
Description: | modified (diff) |
Owner: | set to Evan |
Priority: | highest → normal |
comment:3 Changed 9 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 9 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
Closing since it's due to an inappropriate usage/test case.
comment:5 Changed 9 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 9 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 9 years ago by
Milestone: | 1.8 → 1.6.2 |
---|---|
Resolution: | invalid → duplicate |
Changed 9 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 9 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 9 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.