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

fix plugin header container alignment #6983

Merged

Conversation

DoroNahari
Copy link
Contributor

@DoroNahari DoroNahari commented Jan 28, 2020

Signed-off-by: Doron Nahari doron.nahari@sap.com

What it does

In the plugins view (view -> plugins), when hovering on one of the plugins you can see that the hover color is not applied on all of the plugin container.

This pr fixes the alignment of each plugin container. the hover color fixed accordingly.
You can see the gap changes between the container and the border in the before and after images.

Before:
Screen Shot 2020-01-28 at 0 28 41
Screen Shot 2020-01-28 at 14 55 22

After:
Screen Shot 2020-01-28 at 0 44 13
Screen Shot 2020-01-28 at 14 56 47

How to test

Deploy some plugins to Theia. go to the plugins view (view -> plugins) hover on one of the plugins.

Review checklist

Reminder for reviewers

Signed-off-by: Doron Nahari <doron.nahari@sap.com>
Copy link
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

hover-plugins

👍

@benoitf benoitf added the plug-in system issues related to the plug-in system label Jan 28, 2020
@vince-fugnitto
Copy link
Member

@benoitf do we merge?

@vince-fugnitto
Copy link
Member

Merging :) Thank you for your contribution @DoroNahari 👍

@vince-fugnitto vince-fugnitto merged commit 554f99e into eclipse-theia:master Jan 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plug-in system issues related to the plug-in system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants