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

[plugin] [webview] Add min-height for webview icon as for other icons of tabs #5154

Merged
merged 1 commit into from
May 17, 2019

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented May 15, 2019

Fixes eclipse-che/che-theia#174

webview icon was missing min-height after #4600

related to #4664 as well

https://github.com/theia-ide/theia/blob/4d74d28857cf994da063dac8672272d774c148b0/packages/core/src/browser/style/tabs.css#L136

before patch
image

after patch
image

Change-Id: I0e89d96052cd0b16073478f1d02ba9f309bd23e8
Signed-off-by: Florent Benoit fbenoit@redhat.com

Change-Id: I0e89d96052cd0b16073478f1d02ba9f309bd23e8
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@benoitf benoitf added the plug-in system issues related to the plug-in system label May 15, 2019
@benoitf benoitf requested a review from jbicker May 15, 2019 13:52
@benoitf benoitf requested a review from evidolob as a code owner May 15, 2019 13:52
@benoitf benoitf self-assigned this May 15, 2019
@benoitf
Copy link
Contributor Author

benoitf commented May 17, 2019

@jbicker ?

@jbicker
Copy link
Contributor

jbicker commented May 17, 2019

Oups. Thanks for the reminder! Reviewing it now.

Copy link
Contributor

@jbicker jbicker left a comment

Choose a reason for hiding this comment

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

LGTM

@benoitf benoitf merged commit 15dd61b into master May 17, 2019
@benoitf benoitf deleted the che-theia-174 branch May 17, 2019 08:16
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.

Minor UI alignment issue with Welcome page tab icon
2 participants