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.

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

Test: Multi Diff Editor #199155

Closed
3 tasks done
hediet opened this issue Nov 27, 2023 · 5 comments
Closed
3 tasks done

Test: Multi Diff Editor #199155

hediet opened this issue Nov 27, 2023 · 5 comments

Comments

@hediet
Copy link
Member

hediet commented Nov 27, 2023

Refs: #179000

Complexity: 4

Create Issue


This iteration, we implemented the (still experimental) multi diff editor.

Preparation

  • Open a github repository (e.g. vscode)
  • Install latest pre-release GHPRI extension
  • Set "multiDiffEditor.experimental.enabled": true

Test

How To Open A Multi Diff Editor

Try to open the multi diff editor from various places. The icon is always the same. Test if it is intuitive how to open the view.

Notice that the multi file diff editor does not yet react to files becoming changed/unchanged while the view is open.

  • SCM view (make sure you have multiple locally changed files) - test this for both staged and unstaged changes

Image

  • SCM view: Incoming changes/Outgoing changes (outgoing optional)

  • For Pull Request in the GitHub sidebar

Multi Diff Editor Itself

  • Verify that collapsing files work
  • Verify that scrolling works (sufficiently smooth)
  • Verify that editor options are applied, even if different file types get different settings (e.g. change a ts and a json file, and configure the font size for json files in your settings)
  • Verify that reloading VS Code works while the multi diff editor is open
  • Verify that the scroll position is restored when switching to a different editor in the same tab group and back

Please also try to test the UX of this feature - try reviewing PRs using the multi diff editor! What features do you miss the most? What bugs are the most annoying?

@isidorn isidorn self-assigned this Nov 27, 2023
@isidorn isidorn added this to the November 2023 milestone Nov 27, 2023
@gjsjohnmurray
Copy link
Contributor

gjsjohnmurray commented Nov 27, 2023

It is also openable from Timeline view when this is showing commits (which it does by default). I misremembered.

@bpasero
Copy link
Member

bpasero commented Nov 28, 2023

@gjsjohnmurray does it?

@gjsjohnmurray
Copy link
Contributor

@gjsjohnmurray does it?

Sorry, I had misremembered. Updated my comment.

@bpasero
Copy link
Member

bpasero commented Nov 28, 2023

Pretty cool 👍 . Sometimes I was not able to click into the diff editor anymore (right hand side), but not sure how to reproduce that.

@alexdima
Copy link
Member

👏 Very nice! And of high quality!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants