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

[Sticky Scrolling] Fix separator color on windows #1967

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Christopher-Hermann
Copy link
Contributor

The separator in the sticky lines controls are drawn in a bad color on windows dark theme. With this change, the separator color is aligned with the theme.

Current coloring on windows:
Bildschirmfoto 2024-06-14 um 10 42 22

How it should look like:
Bildschirmfoto 2024-06-14 um 10 43 20

This is a follow up for #1894

The separator in the sticky lines controls are drawn in a bad color on windows dark theme. With this change, the separator color is aligned with the theme.
@Christopher-Hermann
Copy link
Contributor Author

@Wittmaxi could you please check if this solves the issue on windows dark theme? If not, I need to set the correct background color for the separator.

@Christopher-Hermann Christopher-Hermann marked this pull request as draft June 14, 2024 08:45
@BeckerWdf
Copy link
Contributor

This is a follow up for #1894

You meant point "1" in #1894 (comment) ?

@Christopher-Hermann
Copy link
Contributor Author

You meant point "1" in #1894 (comment) ?

Yes, correct. Linking the comment is actually a good idea :)

Copy link
Contributor

Test Results

 1 210 files   1 210 suites   53m 22s ⏱️
 7 661 tests  7 431 ✅ 228 💤 2 ❌
16 096 runs  15 625 ✅ 469 💤 2 ❌

For more details on these failures, see this check.

Results for commit fc9981d.

@Wittmaxi
Copy link
Contributor

Wittmaxi commented Jun 14, 2024

grafik

This is what I see, so no, I Would not consider this a fix. I tried to play around with different color values but I could not find a good fix.
I primarily use dark mode, so I would really appreciate you fixing this :) - thanks for putting in the work!

@Christopher-Hermann
Copy link
Contributor Author

I tried to play around with different color values but I could not find a good fix.

Could you try to set the color for both separator in StickyLinesControl line 180:
bottomSeparator.setBackground(Display.getDefault().getSystemColor(SWT.COLOR_WIDGET_LIGHT_SHADOW));

@Wittmaxi
Copy link
Contributor

@Christopher-Hermann this is precisely what I tried (well, with another color) - it had no effect on the bars for me.

@Christopher-Hermann
Copy link
Contributor Author

@Christopher-Hermann this is precisely what I tried (well, with another color) - it had no effect on the bars for me.

Okay, damn you SWT :D Then I need to check on my virtual machine if I can reproduce the error to find a solution.
But it's really strange, I assume that other separators are colored correctly on Windows dark theme.

@Wittmaxi
Copy link
Contributor

@Christopher-Hermann Yes, I have had regular issues with SWT in the past - the dark theme seems to be especially prone to bugs and inconsistencies ;_;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants