gridstack.js icon indicating copy to clipboard operation
gridstack.js copied to clipboard

could support nest grid more than 2 level

Open joebnb opened this issue 3 years ago • 7 comments

https://github.com/gridstack/gridstack.js/blob/afd7349ec4a0c54847d31703123efd1d037f2548/src/gridstack-dd.ts#L135

if remove this line it's seems almost finish about mutiple level,could implement this.

joebnb avatar Mar 28 '22 08:03 joebnb

2022-03-28 16-57-48 2022-03-28 16_59_00 if comment this line, it will looks like this image,there are 2 problem to use this

incorrect grid width and height and if drop disappear in some case when the drop shadow not show

joebnb avatar Mar 28 '22 09:03 joebnb

I have not tested deep nesting as the current demo only does 1 level and the v5.0 focus was on allowing to move items in/out of that hierarchy - but something I could work on for the right donation, or you can debug and fix the issue which would be great. Posting a demo showing the issue (use nested.html as starting point) would help. thanks.

adumesny avatar Mar 30 '22 14:03 adumesny

are there any updates on this issue? perhaps you managed to find a workaround?

v1talii-dev avatar Jun 06 '22 15:06 v1talii-dev

@joebnb did you manage to solve the problem?

v1talii-dev avatar Jun 08 '22 15:06 v1talii-dev

As far as I was able to identify, the problem is that style classes are not applied to elements in the subgrid.

this is how it looks before the changes: image

and so after dragging: image

but if you immediately render the fields, then it is output correctly: image

however, after that, if you drag the tabs, there will be the same incorrect behavior...

v1talii-dev avatar Jun 10 '22 15:06 v1talii-dev

thanks, but as I mentioned dragging to created new nested grids is a big feature request...

adumesny avatar Jun 14 '22 20:06 adumesny

I found a workaround, it seems to work. @adumesny please see the PR.

v1talii-dev avatar Jun 15 '22 10:06 v1talii-dev

fixed by #2052 for general drag to create sub-grids feature in v7

adumesny avatar Sep 26 '22 14:09 adumesny