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

Red box-colour leaks over toolbar #36

Closed
ingomohr opened this issue Jun 11, 2020 · 7 comments · Fixed by #64
Closed

Red box-colour leaks over toolbar #36

ingomohr opened this issue Jun 11, 2020 · 7 comments · Fixed by #64
Labels
bug Something isn't working

Comments

@ingomohr
Copy link
Contributor

Screenshot 2020-06-11 at 23 48 34

As discussed w/ @AObuchow, we want to have this fixed. :)

@ingomohr ingomohr changed the title Red Box-Colour Leaks Over Toolbar Red box-colour leaks over toolbar Jun 11, 2020
@AObuchow AObuchow added the bug Something isn't working label Jun 11, 2020
AObuchow added a commit that referenced this issue Jun 12, 2020
Fix #34
Fix #13
Fix #40
Part of #36

Signed-off-by: Andrew Obuchowicz <aobuchow@redhat.com>
AObuchow added a commit that referenced this issue Jun 12, 2020
Fix #13
Fix #40
Part of #36

Signed-off-by: Andrew Obuchowicz <aobuchow@redhat.com>
AObuchow added a commit that referenced this issue Jun 12, 2020
Fix #13
Fix #40
Part of #36

Signed-off-by: Andrew Obuchowicz aobuchow@redhat.com
AObuchow added a commit that referenced this issue Jun 12, 2020
Fix #13
Fix #40
Part of #36

Signed-off-by: Andrew Obuchowicz aobuchow@redhat.com
AObuchow added a commit that referenced this issue Jun 12, 2020
Part of #36

Signed-off-by: Andrew Obuchowicz <aobuchow@redhat.com>
@AObuchow
Copy link
Owner

#43 partly fixes this, although when a part is resized the issue appears again.

AObuchow added a commit that referenced this issue Jun 13, 2020
Part of #36

Signed-off-by: Andrew Obuchowicz <aobuchow@redhat.com>
@AObuchow
Copy link
Owner

#43 partly fixes this, although when a part is resized the issue appears again.

I now think that there's an upstream bug that occurs when a view is resized. Will have to make a gif to demonstrate it and file an upstream bug.

@AObuchow
Copy link
Owner

To describe the issue (before getting a gif of it):
When a view that contains a toolbar is resized, all views are also resized (since they all share the common space). It's possible to define a color to use when this resize occurs, however I haven't found a way to define a color to use when resizing and when the view is active.

This issue isn't apparent in the platform dark theme since the tab background colors aren't changed depending on whether they are active or not. This isn't the case for Modern Dark as active tabs get an accent color.

@AObuchow
Copy link
Owner

Here's a gif of the current bug that exists with the toolbar colors. The project explorer toolbar icons background changes to grey upon resizing a view.
bug
(Excuse the low quality gif... I don't know why Giphy made it so small)

This grey coloring (on resize) is coming from here.

In theory, this might be fixable by adding something like:

.MPartStack.active > CTabFolder > Composite#ToolbarComposite {
  background-color: '#org-eclipse-ui-workbench-PRIMARY_COLOR'; 
}

As we want to make the toolbar use the primary color if it's the active view.
However, I'm either not using Eclipse CSS correctly (most likely..) or this requires a fix upstream.

@PyvesB you might have better luck trying to fix this, as you know more about Eclipse CSS than I do :)

@PyvesB
Copy link
Contributor

PyvesB commented Jun 18, 2020

Maybe something like this instead?

CTabFolder.active > Composite#ToolbarComposite {
  background-color: '#org-eclipse-ui-workbench-PRIMARY_COLOR'; 
}

AObuchow added a commit that referenced this issue Jun 18, 2020
Fix #36

Signed-off-by: Andrew Obuchowicz <aobuchow@redhat.com>
@AObuchow
Copy link
Owner

@PyvesB yup that worked :D Thank you!!!

AObuchow added a commit that referenced this issue Jun 18, 2020
Fix #36

Signed-off-by: Andrew Obuchowicz <aobuchow@redhat.com>
@ingomohr
Copy link
Contributor Author

Much better now. Thank you! :) @AObuchow @PyvesB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants