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

repository does not always shows up in the git diff view in a multi root workspace #5999

Closed
elaihau opened this issue Aug 21, 2019 · 2 comments
Assignees
Labels
bug bugs found in the application git issues related to git multi-root issues related to multi-root support

Comments

@elaihau
Copy link
Contributor

elaihau commented Aug 21, 2019

Description

The repository does not always shows up in the git diff view in a multi root workspace.

Reproduction Steps

  • Open a folder as the workspace
  • Add one more folder to make a multi root workspace
  • Right click one folder from the 2nd root -> Compare With ... -> Open the diff view
    • Name of the Repository shows up in the view
  • Right click one folder from the 1st root -> Compare With ... -> Open the diff view
    • Name of the Repository does not show up in the view

Result: Repository shows up in the diff view for one root, not the other
Expected: If the repository shows up for one root, it should also show up for the other root.

See GIF below:

Peek 2019-08-20 22-47

OS and Theia version:
Ubuntu 18.04, example browser, master branch as of Aug 20, 2019.

@elaihau elaihau added the git issues related to git label Aug 21, 2019
@elaihau elaihau changed the title inconsistent git-diff views in multi root workspace repository does not always shows up in the git diff view in a multi root workspace Aug 21, 2019
@akosyakov akosyakov added bug bugs found in the application multi-root issues related to multi-root support labels Aug 21, 2019
@elaihau
Copy link
Contributor Author

elaihau commented Aug 22, 2019

I did some investigation.
Looks like the inconsistency is by design:
https://github.com/theia-ide/theia/blob/c1d367f30333113423689af9344a4c7e8e15579d/packages/git/src/browser/git-navigable-list-widget.tsx#L89

If the diff is not based on the active / selected git repo, the Repository row is rendered, otherwise the row does not show up.

Maybe we should close this one? @jbicker @akosyakov @vince-fugnitto

@elaihau
Copy link
Contributor Author

elaihau commented Aug 23, 2019

I am closing this one, and if there are objections please free feel to reopen.

@elaihau elaihau closed this as completed Aug 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application git issues related to git multi-root issues related to multi-root support
Projects
None yet
Development

No branches or pull requests

2 participants