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

[Feature request] Enable persistance in Cytoscape component #113

Open
ManInFez opened this issue Dec 10, 2020 · 4 comments · May be fixed by #121
Open

[Feature request] Enable persistance in Cytoscape component #113

ManInFez opened this issue Dec 10, 2020 · 4 comments · May be fixed by #121

Comments

@ManInFez
Copy link

Description

Persistence is now being integrated into dash core components, https://dash.plotly.com/persistence. This would be a very useful feature in Cytoscape when dealing with multi-page dash applications. Seems to be small additions to the code, as explained in "For component developers" section on the dash web-page.

@xhluca
Copy link
Collaborator

xhluca commented Feb 4, 2021

That sounds pretty interesting! I'm looking at the PR (plotly/dash-core-components#854) and I'm wondering which props we will want to extract and apply. Other than elements, what would you suggest?

@ManInFez I'll try to work on a PR adding that feature once I have a better understanding, but I can't think of a good demo for showcasing/testing persistence. Is there a specific app you have in mind? If so I'd be happy to review & merge a PR if you are interested in adding it to the demos!

@ManInFez
Copy link
Author

ManInFez commented Feb 4, 2021

I have a multipage app which uses cytoscape as as a selection-widget for the overarching data objects. When switching pages, the selection is lost. It can be partially mitigated by using the Store-component, but since the selectedNodeData component in cytoscape is missing the nodes where selected=True on creation, it is a bit lacking. Also, it complicates the callback structure and creates more data traffic than necessary.

I can try to get a smaller demo set up, to test on :)

@xhluca
Copy link
Collaborator

xhluca commented Feb 4, 2021

Hmm, I guess i'll start with elements, stylesheet and layout in this case.

@ManInFez ManInFez linked a pull request Feb 8, 2021 that will close this issue
6 tasks
@ManInFez
Copy link
Author

ManInFez commented Feb 8, 2021

Here is a test-case #121 @xhlulu

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 a pull request may close this issue.

3 participants
@ManInFez @xhluca and others