vscode-git-graph icon indicating copy to clipboard operation
vscode-git-graph copied to clipboard

"View Diff" should open VS Code's rich diff viewer for Jupyter Notebooks

Open RyanCantab opened this issue 3 years ago • 6 comments

Describe the improvement that you'd like VS Code has a native diff viewer for Jupyter notebook files. This is the default diff view when you select "Show Changes" on a notebook file in the Source Control sidebar. However, Git Graph's "View Diff" feature opens the diff of the raw text file, which is generally not useful. Git Graph should use the same diff view that VS Code uses.

Additional context (optional) This is what a notebook diff looks like: image

RyanCantab avatar Dec 08 '21 15:12 RyanCantab

Can anyone please follow up with this? Thanks

lamwilton avatar Apr 18 '22 17:04 lamwilton

It would be great if someone could take a look at this. Currently, it makes viewing changes in commits in data-science repositories impossible using the git-graph extension.

I have tried to find the issue, but unfortunately, I'm unfamiliar with Javascript / Typescript.

It seems like git-graph executes the command vscode.diff, just like the Open Changes context option from the default source control panel. So the issue is likely in the parameters passed to that command.

JeroenPeterBos avatar Jun 20 '22 15:06 JeroenPeterBos

Can anyone please follow up with this? Thanks @mhutchie

RiccardoRomagnoli avatar Sep 11 '23 13:09 RiccardoRomagnoli

I don't have time to actually try this out, but after a bit of digging it looks like git-graph is passing the diff command URIs with 'git-graph' as the scheme (https://github.com/mhutchie/vscode-git-graph/blob/develop/src/diffDocProvider.ts#L140), whereas the built-in source control pane uses 'git' as the scheme (https://github.com/microsoft/vscode/blob/main/extensions/git/src/uri.ts#L30)

tmchartrand avatar Sep 11 '23 23:09 tmchartrand

Please follow up @mhutchie 👍

SCantergiani avatar Feb 07 '24 18:02 SCantergiani