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

[git diff] fix the git-diff widget header details alignment #5998

Merged
merged 1 commit into from
Aug 21, 2019

Conversation

vince-fugnitto
Copy link
Member

@vince-fugnitto vince-fugnitto commented Aug 20, 2019

What it does

Fixes #5996

  • fixes the git-diff widget header details but updating the css
    specifically for the git-diff widget header. Originally the path and
    revision details were not aligned properly and it has been resolved
    so that they are displayed inline with their values.
Before After
Screen Shot 2019-08-20 at 7 49 38 PM Screen Shot 2019-08-20 at 7 46 28 PM

How to test

  1. open the application with a given workspace (example: theia)
  2. right-click on the packages/ folder in the explorer and select Compare With...
  3. when prompted, select a branch
  4. verify that the alignment for path, revision and for files changed is correct in the git-diff header

Review checklist

Reminder for reviewers

Signed-off-by: Vincent Fugnitto vincent.fugnitto@ericsson.com

Fixes #5996

- fixes the `git-diff` widget header details but updating the css
specifically for the `git-diff` widget header. Originally the `path` and
`revision` details were not aligned properly and it has been resolved
so that they are displayed inline with their values.

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
@vince-fugnitto vince-fugnitto added git issues related to git ui/ux issues related to user interface / user experience labels Aug 20, 2019
@vince-fugnitto vince-fugnitto self-assigned this Aug 20, 2019
Copy link
Contributor

@elaihau elaihau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While testing I found #5999 but I don't think it is related to this pull request.

This change works great in both single and multi root workspace.
Also, I tried different width and height and the appearance of the git-diff view was not broken.

Thank you for your work !

@vince-fugnitto
Copy link
Member Author

@elaihau thank you for the review! I'll wait a while to see if anyone else has any comments :)

@vince-fugnitto
Copy link
Member Author

@elaihau thank you for the review!

@vince-fugnitto vince-fugnitto merged commit c1d367f into master Aug 21, 2019
@vince-fugnitto vince-fugnitto deleted the vf/GH-5996 branch August 21, 2019 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
git issues related to git ui/ux issues related to user interface / user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[git diff] adjust the alignment of the git diff widget header details
2 participants