Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

It's nearly impossible to tell where a file has moved from and to #1961

Open
jcansdale opened this issue Sep 28, 2018 · 2 comments
Open

It's nearly impossible to tell where a file has moved from and to #1961

jcansdale opened this issue Sep 28, 2018 · 2 comments
Labels

Comments

@jcansdale
Copy link
Collaborator

jcansdale commented Sep 28, 2018

Version

  • GitHub Extension for Visual Studio version: 2.5.6 (dev)
  • Visual Studio version: All

What happened

Steps to Reproduce

  1. Open a PR with moved file in PR details view (e.g. Refactor: Move views to GitHub.VisualStudio.UI #1955)
  2. Double click on a moved file

Expected behavior: [What you expect to happen]

Tooltip of diff tab shows source and destination of moved file.

Actual behavior: [What actually happens]

Tooltip shows destination path as both source and destination.
File appears as an added file in diff (all green on RHS).

Screenshot or GIF:

image

image

If I try to comment, I get position is invalid:
image

Here is the same file on dotcom for comparison:
image

Proposed Solution

  1. Fix tooltip and diff on diff view
  2. Show similar source and destination tooltop on tree view

Having just the destination on the tree view node isn't useful without a source to compare it with. I'd suggest simply showing [moved] on the node to avoid horizontal scrolling issues for minimal gain. Having the a source and destination as the tooltip would be very useful.

image

@jcansdale jcansdale added the bug label Sep 28, 2018
@jcansdale
Copy link
Collaborator Author

@meaghanlewis could you see if you can repro this? It's a bit of a nasty one. 😭

@meaghanlewis
Copy link
Contributor

@jcansdale I can reproduce this and see exactly what you mean about not being able to tell a file was moved and not being able to comment on the moved file.

I don't see any other editors handle this situation. I think it makes sense yo try to do what dotcom does here like you suggest and:

  1. fix the tooltip to show the source and destination
  2. fix the diff on diff view to show "File renamed without changes" to prevent commenting

@github-project-automation github-project-automation bot moved this to Medium Priority in BUGS Aug 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
Archived in project
Development

No branches or pull requests

2 participants