Opened 6 years ago

Closed 3 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: cbaker118@…

Change History (2)

comment:1 Changed 6 years ago by Patrick Ruzand

Owner: changed from Eric Durocher to Patrick Ruzand
Status: newassigned

comment:2 Changed 3 years ago by dylan

Milestone: tbd1.12
Resolution: patchwelcome
Status: assignedclosed

Given that no one has shown interest in creating a patch in the past 2+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.