Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support nest grid more than 2 level #1943

Closed
joebnb opened this issue Mar 28, 2022 · 8 comments
Closed

support nest grid more than 2 level #1943

joebnb opened this issue Mar 28, 2022 · 8 comments

Comments

@joebnb
Copy link

joebnb commented Mar 28, 2022

if (node?.subGrid) return false;

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

@joebnb
Copy link
Author

joebnb commented Mar 28, 2022

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

@adumesny
Copy link
Member

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.

@v1talii-dev
Copy link
Contributor

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

@v1talii-dev
Copy link
Contributor

@joebnb did you manage to solve the problem?

@v1talii-dev
Copy link
Contributor

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...

@adumesny
Copy link
Member

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

@v1talii-dev
Copy link
Contributor

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

@adumesny adumesny changed the title could support nest gird more than 2 level could support nest grid more than 2 level Jul 1, 2022
@adumesny
Copy link
Member

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

@joebnb joebnb changed the title could support nest grid more than 2 level support nest grid more than 2 level Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants