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

Dependency Tree branch once opened does not retract back on closing #182

Closed
AdiAkhileshSingh15 opened this issue Mar 9, 2024 · 2 comments · Fixed by #186
Closed

Dependency Tree branch once opened does not retract back on closing #182

AdiAkhileshSingh15 opened this issue Mar 9, 2024 · 2 comments · Fixed by #186
Assignees
Labels
bug Something isn't working

Comments

@AdiAkhileshSingh15
Copy link
Contributor

AdiAkhileshSingh15 commented Mar 9, 2024

Description

This issue aims to make the Dependency Tree Structure Interface tidy and readable to user.

Screenshot from 2024-03-08 21-43-23
When the app branch is opened.

Screenshot from 2024-03-08 21-43-56
When the app branch is closed.

Here, it leaves the tree skeleton upto what it was opened before that point, behind the current tree structure, which I don't think serves any purpose and leaves the UI pretty cluttered.
I'm trying to find a fix for this, as I was going through the ReactEcharts module props for tree type.
I'd be happy to take this issue up, if it is one.

@lrasmus Please take a look a it.

@lrasmus lrasmus added the bug Something isn't working label Mar 11, 2024
@lrasmus
Copy link
Contributor

lrasmus commented Mar 11, 2024

@AdiAkhileshSingh15 - thank you for pointing this out, I agree it's an error! Assigned to you, if you're willing to investigate.

@AdiAkhileshSingh15
Copy link
Contributor Author

Update

While trying to fix this, I discovered what's causing this issue is the data object. What I noticed is that it works fine if I give it the data object in the same form but hard-coded, than using the StatWrap methods to generate the data. I don't understand why, even if I deconstruct the data object into just name and children, which is how the docs for echarts-for-react do, it doesn't work, or even passing it through a wrapper function to transform data doesn't help. So, I just don't understand what exactly is wrong with the data object. But, am sure what's causing the issue is definitely data and not the other attributes of option prop, as it works fine for hard-coded data as I said.

Potential Issue:
What may actually be causing the issue, as I discussed with @lrasmus, is the complexity of the data generated by StatWrap, or the inability of the rendering package to deal with such complex data.
I raised this issue in the parent repository of this package, which you can find here. Still waiting for the response though :)) Dunno, maybe upvotes or smthg can work.

Way forward:
So, am further investigating it if that's the real issue by passing in some complex hard-coded data. And on confirming that, will look for new packages which can come of some help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants