feat: include relative path as argument to render each node in tree view #45
Conversation
joaoassisb
changed the title
| @@ -53,9 +53,9 @@ export function TreeViewExamples() { | |||
| return ( | |||
| <TreeView | |||
| nodes={nodes} | |||
| template={(item, index, parentItem) => ( | |||
| template={(item, index, parentItem, relativePath) => ( | |||
There was a problem hiding this comment.
the idea here is to facilitate the usage of the TreeView Component with Forms.
| <em> | ||
| {item.attrA} (index: {index} parent: {parentItem?.attrA || <em>None</em>}) | ||
| {item.attrA} (index: {index} parent: {parentItem?.attrA || <em>None</em>})<p>Path: {relativePath}</p> |
There was a problem hiding this comment.
as the tree grows, only de index and the father element are not enough to handle changes in deep nodes.
With this new argument, change node values is a lot easier.
package.json
Outdated
| @@ -5,7 +5,7 @@ | |||
| "main": "dist/main.js", | |||
| "scripts": { | |||
| "build": "rollup --config", | |||
| "start": "webpack-dev-server --mode development", | |||
There was a problem hiding this comment.
this code was launching this error
package.json
Outdated
| @@ -5,7 +5,7 @@ | |||
| "main": "dist/main.js", | |||
| "scripts": { | |||
| "build": "rollup --config", | |||
| "start": "webpack-dev-server --mode development", | |||
| "start": "webpack serve --mode development", | |||
There was a problem hiding this comment.
after a little research a found this solution
There was a problem hiding this comment.
Please, move this change to another PR, because it is not related to the feature proposed here
webpack.config.js
Outdated
| @@ -28,7 +28,7 @@ module.exports = [ | |||
| { | |||
| entry: './demo/demo.jsx', | |||
| output: { | |||
| path: `${__dirname}/demo`, | |||
| path: `${__dirname}/`, | |||
There was a problem hiding this comment.
The index.html imports the file from ./demo.js not ./demo/demo.js
There was a problem hiding this comment.
Please, move this change to another PR, because it is not related to the feature proposed here
package.json
Outdated
| @@ -5,7 +5,7 @@ | |||
| "main": "dist/main.js", | |||
| "scripts": { | |||
| "build": "rollup --config", | |||
| "start": "webpack-dev-server --mode development", | |||
| "start": "webpack serve --mode development", | |||
There was a problem hiding this comment.
Please, move this change to another PR, because it is not related to the feature proposed here
webpack.config.js
Outdated
| @@ -28,7 +28,7 @@ module.exports = [ | |||
| { | |||
| entry: './demo/demo.jsx', | |||
| output: { | |||
| path: `${__dirname}/demo`, | |||
| path: `${__dirname}/`, | |||
There was a problem hiding this comment.
Please, move this change to another PR, because it is not related to the feature proposed here
No description provided.