#17212 closed defect (fixed)
TreeMap drill up is broken
Reported by: | stelian56 | Owned by: | cjolif |
---|---|---|---|
Priority: | undecided | Milestone: | 1.10 |
Component: | Dojox | Version: | 1.9.0 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Steps to reproduce:
- Open dojox/treemap/tests/test.html
- Select Group by: Mode then State
- Double click Water to drill down to level 2
- Double click CA to drill down to level 3
- Double click CA to drill up to level 2
The tree map jumps all the way to level 1 instead.
Suggested fix to dojox/treemap/TreeMap.js:
refreshRendering: function(){ ... var parentItem = null; var rootItemRenderer = this._getRenderer(rootItem); if (rootItemRenderer) { parentItem = rootItemRenderer.parentItem; } this._buildRenderer(this.domNode, parentItem, rootItem, { ...
Change History (6)
comment:1 Changed 8 years ago by
Component: | General → Dojox |
---|---|
Owner: | set to cjolif |
Status: | new → assigned |
comment:2 Changed 8 years ago by
comment:3 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:4 Changed 8 years ago by
Christophe,
Thank you for fixing the bug.
I have just signed an individual CLA. However, I am reluctant to become a contributor. If you don't mind, I am going to suggest any future bug fixes via bug reports.
Stelian
comment:6 Changed 8 years ago by
Milestone: | tbd → 1.10 |
---|
Note: See
TracTickets for help on using
tickets.
Thanks for reporting this.
I will commit a slightly different fix (but same spirit). For the future stelian56 please make sure to sign a Dojo CLA even for fixes contributions (maybe you have one but not listed as stelian56). See: http://dojofoundation.org/about/claCheck and now: https://github.com/dojo/dojox/blob/master/CONTRIBUTING.md for more details on how we expect contributions now that we moved to github.