Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#9501 closed enhancement (fixed)

[cla][patch] TreeGrid Supports N-level

Reported by: Mina R Waheeb Owned by: Nathan Toone
Priority: high Milestone: 1.4
Component: DojoX Grid Version: 1.3.0
Keywords: Tree Grid Cc:
Blocked By: Blocking:

Description

Current implementation of TreeGrid? support only predefined levels. The attached implementation adds support for N-Level Grids which is very common case.

Thanks

Attachments (2)

TreeGrid.js (8.1 KB) - added by Mina R Waheeb 10 years ago.
B-Level Tree Grid implementation
TreeGrid.html (5.2 KB) - added by Mina R Waheeb 10 years ago.
The test case

Download all attachments as: .zip

Change History (14)

Changed 10 years ago by Mina R Waheeb

Attachment: TreeGrid.js added

B-Level Tree Grid implementation

Changed 10 years ago by Mina R Waheeb

Attachment: TreeGrid.html added

The test case

comment:1 Changed 10 years ago by Nathan Toone

Milestone: tbd1.4
Owner: changed from Bryan Forbes to Nathan Toone
Summary: TreeGrid Supports N-level[cla][patch] TreeGrid Supports N-level

Contributed under CLA for Kemetex Corporation - sent in 7/4/09

comment:2 Changed 10 years ago by Nathan Toone

Resolution: fixed
Status: newclosed

(In [18736]) Fixes #9501 - import patch to allow specifying a treeModel on a TreeGrid? in order to have a "Columnar Tree" as opposed to just an "Expandable Grid" !strict

comment:3 Changed 10 years ago by Nathan Toone

(In [20052]) Refs #9501 - remove rows when closing so that the alternating row colors are displayed correctly !strict

comment:4 Changed 10 years ago by Bryan Forbes

(In [20053]) refs #9501 !strict

  • Changed second argument to dojo.map to a function for speed.
  • Changed a dojo.forEach to .concat([]) in _TreeView.js.
  • Added _onNew to TreeGrid? to keep from adding child nodes to the root. Still need to get adding to an open node working.
  • Added _addItem to TreeGrid? to add the root items to the model.
  • Changed _setStore in TreeGrid? to add this.treeModel.root.children since we don't use the model's querying.
  • Added buttons to test_treegrid_model.html to do adding and deleting.

comment:5 Changed 10 years ago by Nathan Toone

(In [20054]) refs #9501 - add in test case for defaultOpen=true !strict

comment:6 Changed 10 years ago by Bryan Forbes

(In [20055]) * Added "Delete California" button (refs #9501 !strict).

comment:7 Changed 10 years ago by Bryan Forbes

(In [20057]) * Added programmatic creation example (refs #9501 !strict).

comment:8 Changed 10 years ago by Nathan Toone

(In [20062]) Refs #9501 - fix programmatic creation syntax !strict

comment:9 Changed 10 years ago by Nathan Toone

(In [20067]) Refs #9501 - fix notification and defaultOpen handling of grid, update test case to be more robust (no failures when adding the same item), defaultOpen defaults to false for model-based grids, and true for non-model-based treegrids !strict

comment:10 Changed 10 years ago by Nathan Toone

(In [20073]) Refs #9501 - fix rendering so that we are not using timeouts, and it is more performant. Model-based now behaves more like non-model-based. Row alternating behaves like non-model-based again... !strict

comment:11 Changed 10 years ago by Nathan Toone

(In [20083]) Refs #9501 - add in "tree-like" indicators and styling so that it is a bit easier to follow. !strict

comment:12 Changed 10 years ago by Nathan Toone

(In [20085]) Refs #9501 more minor styling changes - no vertical bar

Note: See TracTickets for help on using tickets.