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.

Sign up for GitHub

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

Jump to bottom

Allow graph exports #7884

Open
dmosesson opened this issue Nov 1, 2024 · 7 comments
Open

Allow graph exports #7884

dmosesson opened this issue Nov 1, 2024 · 7 comments
Labels
enhancement This is the preferred way to describe new end-to-end features. waiting external It requires additional info to progress. For example, it can require a fix in other project.

Comments

@dmosesson
Copy link

What do you want to improve?

Allow exports of the entire graph that would be should at the current zoom level if the screen was infinitely bit.

We have people that want to have networking references for our cluster/s, and even if they have to pan around, it's helpful to have an SVG somewhere we can put on confluence, print on extra extra extra large paper, etc.

What is the current behavior?

To capture the graph visually, you need to do a screen capture

What is the new behavior?

Add a way for a user to export the entirety of the graph as it would be displayed if the screen was infinitely big (i.e. preserve the font, edges, boxes, and other settings as they are for the export).
@dmosesson dmosesson added the enhancement This is the preferred way to describe new end-to-end features. label Nov 1, 2024
@jshaughn
Copy link
Collaborator

jshaughn commented Nov 1, 2024

linking a past request for something similar: #4615

@dmosesson
Copy link
Author

Looking into this more, I think cytoscape supports exporting as png and jpg with minimal effort: https://js.cytoscape.org/#core/export

I would be happy with something at least and the options for the png and jpg options are the same with the exception of the quality parameter. How hard would be be to expose this as a button/menu option somewhere on the UI? Would such a path be supported even if it were easy?

@jmazzitelli
Copy link
Collaborator

cytoscape supports exporting

Just in time for Kiali's deprecation of cytoscape and move to PatternFly :)

@jshaughn perhaps we should submit an enhancement request to the PatternFly folks (or maybe they already support this?).

@jshaughn jshaughn added the refinement Still being full defined before taking any action label Nov 4, 2024
@jshaughn
Copy link
Collaborator

jshaughn commented Nov 4, 2024

@jmazzitelli I'm poking around and asking in PFT Slack. There is code that can export what's on-screen, such that you don't have to do a screen shot yourself. But, I don't think a full export of what's on and off-screen exists, now do I know if it's really feasible. Will keep asking around...

@dmosesson
Copy link
Author

Any update on this from the PFT people?

@jshaughn
Copy link
Collaborator

Any update on this from the PFT people?

No obvious solution, right now. One more suggestion to try before I add it is an RFE for PFT topology. Still not sure it's possible.

@jshaughn jshaughn mentioned this issue Dec 2, 2024
Open
@jshaughn
Copy link
Collaborator

jshaughn commented Dec 2, 2024

patternfly/react-topology#260

@jshaughn jshaughn added waiting external It requires additional info to progress. For example, it can require a fix in other project. and removed refinement Still being full defined before taking any action labels Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement This is the preferred way to describe new end-to-end features. waiting external It requires additional info to progress. For example, it can require a fix in other project.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jmazzitelli @jshaughn @dmosesson