Opened 9 years ago
Closed 7 years ago
#16618 closed defect (patchwelcome)
dojox/mobile/TreeView doesn't work with ScreenSizeAware Splitters
Reported by: | Craig Baker | Owned by: | Patrick Ruzand |
---|---|---|---|
Priority: | undecided | Milestone: | 1.13 |
Component: | DojoX Mobile | Version: | |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
In the current nightly code (1.9.0 rev 29469)
In dojox/mobile/TreeView.js
In the function handleClick
When the user clicks on an item, the TreeView? creates a new view, adds items from the store to it, and then transitions to the new view from the current one. This works great with the TreeView_test (seen here http://archive.dojotoolkit.org/nightly/dojotoolkit/dojox/mobile/tests/test_TreeView.html), but there is no container or splitter in that test.
The TreeView? code adds the newly created view to win.body()
var newView = ScrollableView?({
id: newViewId
}, domConstruct.create("div", null, win.body()));
This means that if you're using a ScreenSizeAware? splitter (which errors out if it cannot find 2 containers to render in), the TreeView? will never transition off of the root of the tree, because the newly created view isn't in the container that the current TreeView? widget is in.
The TreeView? code needs to add it to the container that the TreeView? widget currently resides in, if there is one, and if not then it can default to adding it to win.body().
CC: [email protected]…
Change History (2)
comment:1 Changed 9 years ago by
Owner: | changed from Eric Durocher to Patrick Ruzand |
---|---|
Status: | new → assigned |
comment:2 Changed 7 years ago by
Milestone: | tbd → 1.12 |
---|---|
Resolution: | → patchwelcome |
Status: | assigned → closed |
Given that no one has shown interest in creating a patch in the past 2+ years, I'm closing this as patchwelcome.