feat: Don't use Diff when comparing empty media file

Lennard Sprongrequested to merge
gitlab-community/gitlab-org/gitlab-vscode-extension:X_Sheep/image-diff-open into main

Description

Files in a Merge Request will now check if the extension is an image/audio/video file, and will call vscode.open instead of vscode.diff if the file is new or deleted.

Review URLs now also get handled by the FileDecorationProvider so it's easier to distinguish added files from deleted files when opening the preview.

Related Issues

Resolves #1220 (closed)

How has this been tested?

This can be tested with the test MR viktomas/test-project!6 (closed)

Screenshots (if appropriate):

Notice the tab title decoration as well.

Image diff

before after
Screenshot_2024-01-25_at_12.13.55 Screenshot_2024-01-25_at_12.12.00
Screenshot_2024-01-25_at_12.14.01 Screenshot_2024-01-25_at_12.12.07

Decorator showing in the tab title

before after
Screenshot_2024-01-25_at_12.14.17 Screenshot_2024-01-25_at_12.12.32

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)
  • Test gap
Edited by Tomas Vik (OOO back on 2026-01-05)

Merge request reports