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

modern light theme: editor right bar to bright #2201

Open
jukzi opened this issue Aug 16, 2024 · 6 comments
Open

modern light theme: editor right bar to bright #2201

jukzi opened this issue Aug 16, 2024 · 6 comments
Labels
bug Something isn't working

Comments

@jukzi
Copy link
Contributor

jukzi commented Aug 16, 2024

Only the editor content should have white background, but now also the bar right to the scroll bar has white contrast.
image
old scheme had had it grey:
image

@jukzi jukzi added the bug Something isn't working label Aug 16, 2024
@BeckerWdf
Copy link
Contributor

see #2144
for this.
Maybe you can help @mvm-sap with your knowledge to fix the remaining issues in that PR.

@jukzi
Copy link
Contributor Author

jukzi commented Aug 16, 2024

Maybe you can help @mvm-sap with your knowledge to fix the remaining issues in that PR.

Sorry i have no knowledge about how to use themes at all. All i remember is that i did consider the css enginge broken when i looked into it years ago as it did not consider subclasses to match super classes and would suggest to fix that before working with it. Also i would expect that the Layout Spy (shift-ctrl-alt-f9) shows which css classes an element does adhere to. But i can't see helpful information there. Maybe improve that tool first?
image

@jukzi
Copy link
Contributor Author

jukzi commented Aug 28, 2024

with css spy fixed (eclipse-pde/eclipse.pde#1381) one can see this in the css spy (old light theme):
image
and with preview light theme:
image

@jukzi
Copy link
Contributor Author

jukzi commented Aug 28, 2024

i.e. this CSS entry did the coloring:

CTabFolder Canvas {
	background-color: #F8F8F8;
}

Adding that to e4_preview_win.css solves this problem - but i don't know what else it will color too.

@BeckerWdf
Copy link
Contributor

i.e. this CSS entry did the coloring:

CTabFolder Canvas {
	background-color: #F8F8F8;
}

Adding that to e4_preview_win.css solves this problem - but i don't know what else it will color too.

@mvm-sap: Can you pls. have a look at this?

@mvm-sap
Copy link
Contributor

mvm-sap commented Sep 23, 2024

This change will not cause problems in other places. I shall create a PR soon for this.

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

No branches or pull requests

3 participants