Skip to content

Update react-ui-tree.js#14

Open
dannav wants to merge 3 commits into
swiftcarrot:gh-pagesfrom
dannav:patch-1
Open

Update react-ui-tree.js#14
dannav wants to merge 3 commits into
swiftcarrot:gh-pagesfrom
dannav:patch-1

Conversation

@dannav

@dannav dannav commented Oct 17, 2015

Copy link
Copy Markdown

I don't think this is necessary. I may want to move a node under another if it's collapsed. Also this prevents moving any nodes without children under another in the same tree node.

I don't think this is necessary. I may want to move a node under another if it's collapsed. Also this prevents moving any nodes without children under another in the same tree node.
allow creation of static tree to prevent manipulation of structure if defined
@wangzuo

wangzuo commented Oct 19, 2015

Copy link
Copy Markdown
Member

Is this the same problem with issue #13 ?

@dannav

dannav commented Oct 19, 2015

Copy link
Copy Markdown
Author

I'm sorry I should have been more explicit. Yes this change allows you to add nodes to leaf nodes so it should fix that user's problem as well.

@dannav

dannav commented Oct 19, 2015

Copy link
Copy Markdown
Author

I've also added a prop to define a tree as static to prevent manipulation if a user just wants to display the structure which would resolve issue #6

@wangzuo

wangzuo commented Oct 20, 2015

Copy link
Copy Markdown
Member

Whats the new feature for this pr?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants