vscode-vega-viewer icon indicating copy to clipboard operation
vscode-vega-viewer copied to clipboard

add live vega spec preview?

Open RandomFractals opened this issue 6 years ago β€’ 10 comments

this is already set up in extension.ts workspace.onDidChangeTextDocument event handler, but I disabled it for alpha.

If updating preview on json spec save is not sufficient I can enable this feature in the next viewer release by adding some 1 min or so timer (configurable to your liking) for that whole json parser and render process not to fire on every char change (why I disabled it for now)

RandomFractals avatar Jan 08 '19 17:01 RandomFractals

I have not seen any requests for this feature and I don't like taxing code with even once a minute json spec updates that will most likely break a preview until they are complete.

so... closing this for now. save your json spec updates to refresh Vega preview.

RandomFractals avatar Feb 07 '19 14:02 RandomFractals

I was just about to request this feature πŸ˜€. Perhaps the backend can do the following before updating the UI?:

  • watch the file for changes
  • validate as JSON
  • vega.parse for errors

If the above check out, finally render?

danmarshall avatar Nov 21 '19 20:11 danmarshall

might do all of the above in the next v. update ...

I'll reopen this for us to explore more :)

RandomFractals avatar Nov 21 '19 20:11 RandomFractals

Great, thanks for re-opening!

Also, I didn't see the I need to rename my files to .vg.json for the refresh on save. Any chance on cracking the file and looking at $schema? Sorry to ask in this issue.

danmarshall avatar Nov 21 '19 20:11 danmarshall

yeah, Vega dev team pretty much set that standard for naming Vega and Vega-Lite spec json files in their examples that VV exposes too.

I'd rather we keep it that way and use $shema from vega(s) json specs only to decided what vega v. lib to load down the road. see #42 for more info on that. I need to add more notes from my slacks with Vega team to use schema ref. as well.

If you are using or expsoing Vega(s) json specs in SandDance too, I think it would be great if you switched to that json files naming convention too ...

RandomFractals avatar Nov 21 '19 20:11 RandomFractals

in short, VV looks for those extra vg/vl json filename extensions. try 'em! & all the examples that come with it :)

see this section in docs on that part for more info: https://github.com/RandomFractals/vscode-vega-viewer#vega-viewer-vscode-contributions

RandomFractals avatar Nov 21 '19 20:11 RandomFractals

Great to know, I'm on board and will change my extensions to vg & vl πŸ˜€πŸ‘

danmarshall avatar Nov 21 '19 20:11 danmarshall

cool. yeah, use vg.json and vl.json for now. I threw .vg and .vl in the mix in my ext. just for fun. those might go away since we do watn vscode to treat them like any json files and for json schema validation to kick in without much lifting on our end.

RandomFractals avatar Nov 21 '19 20:11 RandomFractals

Right, yes I meant .vg.json & .vl.json - I still need the .json for editing πŸ˜‰

danmarshall avatar Nov 21 '19 20:11 danmarshall

@danmarshall let me know what you think of VV & that woman in the red dress :)

Hopefully, new tweet on VV & SandDance integration is forthcoming ...

by the by, I keep a steady dev log on VV on twitter. hit us up there: https://twitter.com/hashtag/vegaviewer?f=tweets&vertical=default&src=hash

RandomFractals avatar Nov 21 '19 21:11 RandomFractals