Extend output file format and folder selection #9
Comments
|
I could add a multiple-choice option for the user to select other formats to generate in the out dir, though I'm not sure how much it will impact the generation time (wouldn't be worse than at 0.3 when it was generating all of them). Also thinking we should always generate the SVG to preview on the panel (eventually may even not be needed if/when WV implements direct STDOUT output). There's no more need to preview PNG on the panel, as it was only introduced as a workaround for a bug in the SVG output (#3) fixed by 0.4. Note: HTML is just an SVG wrapped inside a |
|
@nanangp wrote:
Sounds okay to me, though only a single file would be needed to preview. I would prefer the speedy way. You might add a simple option to append a string to the command line. If users want all files to be generated, they could simply request it there.
The users should still have a choice on what to display by default. Maybe let them easily switch without opening the extension settings. The question is: Would the picture be regenerated whenever the user is switching? Or would you pre-generate different files and let the user choose?
Some users would still prefer PNG because it is supported by most applications.
With the addition of a BOM and title block, as in Demo02. The output folder for the preview could be temporary, especially in the case that users open files from a read-only location. Would you also add an option to delete the generated files after closing? |
martinrieder
changed the title
I think we're overcomplicating this. All I'm saying is, for the preview panel we'll only use the SVG. If they want a PNG, HTML, GV, whatever output, they can specify it in the optional "other formats to generate" setting mentioned. We just won't be previewing them in the panel. I never envisioned this extension to do output file management, with it being a previewer. The original idea was to not output anything and simply use |
|
Closing this with v1.1.0 that should now allow users to specify values for the command-line args. |
|
I fully agree with you on the point that a preview extension should be kept simple. Thanks for hearing my suggestions though... |
|
Haha yeah, that post was what got me started. I used it for about a year. This extension was an exercise of, "Never done a vscode extension before, let's see how far we can push it". Also a first foray into the Node/Typescript ecosystem. |
|
@nanangp posting into this closed issue to simply let you know of an idea I am pursuing: The HTML output could be enriched by https://github.com/mountainstorm/jquery.graphviz.svg EDIT alternative https://github.com/w8r/leaflet-schematic I am mostly missing the zoom functionality, though this provides a few other options as well. Taking the thought a bit further, it would allow nodes or edges to be highlighted, in case that WireViz reports an issue. |
Please extend the option for selecting the output format and folder. Especially the GV file is not generated by default since WV 0.4, though it might be required for debugging.
It seems sufficient to me if only a single file is generated and displayed in the preview panel. Even HTML output could be shown instead of the pictures.
Since this extension is called "wireviz-preview", it may be an option to generate the files in some temporary folder that might be deleted after closing the file.
The text was updated successfully, but these errors were encountered: