Bug 574054 - Highlight issues with Windows 7 and DevStyle (Darkest Dark)
Summary: Highlight issues with Windows 7 and DevStyle (Darkest Dark)
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.19   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-SWT-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 577357
  Show dependency tree
 
Reported: 2021-06-07 10:19 EDT by Wayne Baldwin CLA
Modified: 2022-01-18 09:16 EST (History)
6 users (show)

See Also:


Attachments
Picture showing how unreadable highlighted items are (153.48 KB, image/jpeg)
2021-06-07 10:19 EDT, Wayne Baldwin CLA
no flags Details
Unreadable dark theme selections in Windows 7 (92.05 KB, image/png)
2021-06-08 02:41 EDT, Ed Merks CLA
no flags Details
Debug View highlights fine (32.18 KB, image/jpeg)
2021-06-09 12:14 EDT, Wayne Baldwin CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Wayne Baldwin CLA 2021-06-07 10:19:52 EDT
Created attachment 286536 [details]
Picture showing how unreadable highlighted items are

When I upgraded Eclipse to the latest version I started having this issue seen in the attached screenshot. It used to work fine with 2020-03 but with 2021 versions you cannot read highlighted items in package explorer, Ant, and History views (among others).

The DevStyle people said it was out of their control, it was an Eclipse bug with the latest versions running on Windows 7.

How do I change that background (or foreground) color so that it's not grey on grey?
Comment 1 Lars Vogel CLA 2021-06-07 11:06:53 EDT
Wayne, do you see the same issue in the default dark theme of the platform?
Comment 2 Thomas Wolf CLA 2021-06-07 16:11:41 EDT
Reported originally in the forums at [1], where Ed Merks wrote he saw the same problem with the default dark theme on Windows 7.

[1] https://www.eclipse.org/forums/index.php/t/1108145/
Comment 3 Ed Merks CLA 2021-06-08 02:41:18 EDT
Created attachment 286543 [details]
Unreadable dark theme selections in Windows 7
Comment 4 Lars Vogel CLA 2021-06-08 02:42:53 EDT
I don't think this color can be styled, moving to SWT.
Comment 5 Lars Vogel CLA 2021-06-08 02:44:08 EDT
Is Win7 still supported my MS?
Comment 6 Ed Merks CLA 2021-06-08 02:53:43 EDT
No, Windows 7 is end of life.
Comment 7 Rolf Theunissen CLA 2021-06-08 08:34:49 EDT
The dark theme issues are not only for Windows 7, on Windows 10 there are many known issues w.r.t selection and the dark theme too: Bug 564960, Bug 566280, Bug 566281
Comment 8 Wayne Baldwin CLA 2021-06-08 10:09:55 EDT
What is SWT?
Comment 9 Thomas Wolf CLA 2021-06-08 10:46:50 EDT
(In reply to Wayne Baldwin from comment #8)
> What is SWT?

The widget toolkit Eclipse uses. https://www.eclipse.org/swt/ .
Comment 10 Rolf Theunissen CLA 2021-06-08 11:03:32 EDT
(In reply to Lars Vogel from comment #4)
> I don't think this color can be styled, moving to SWT.

Request for CSS customization was in Bug 434309, it was implemented (and marked as greatfix). However, it is removed in Bug 562865 (together with broken custom arrow code).
Comment 11 Lars Vogel CLA 2021-06-08 11:14:08 EDT
(In reply to Rolf Theunissen from comment #10)
> (In reply to Lars Vogel from comment #4)

> Request for CSS customization was in Bug 434309, it was implemented (and
> marked as greatfix). However, it is removed in Bug 562865 (together with
> broken custom arrow code).

Did you test if the fix from https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=f014a8c0f49b656bc40d5cd1d25c5b7a81bf2630 works without bad side effects? IIRC it was removed because the custom drawing was creating blurry text in multiple dialogs / popups. Also the creator of the fix was not reacting to request on this custom draing code. So whoever wants to restore this code should also be ready to look into bug reports in this area.

Bug 566281 sounds like an issue with Windows SWT and not something we should try to overpaint in the CSS engine.
Comment 12 Wayne Baldwin CLA 2021-06-08 17:00:41 EDT
There are actually TWO colors here, background and foreground, are both controlled by SWT? How can so many other colors be customized but NOT selected items?
Comment 13 Rolf Theunissen CLA 2021-06-09 02:35:32 EDT
(In reply to Lars Vogel from comment #11)
> (In reply to Rolf Theunissen from comment #10)
> > (In reply to Lars Vogel from comment #4)
> 
> > Request for CSS customization was in Bug 434309, it was implemented (and
> > marked as greatfix). However, it is removed in Bug 562865 (together with
> > broken custom arrow code).
> 
> Did you test if the fix from
> https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/
> ?id=f014a8c0f49b656bc40d5cd1d25c5b7a81bf2630 works without bad side effects?
> IIRC it was removed because the custom drawing was creating blurry text in
> multiple dialogs / popups. Also the creator of the fix was not reacting to
> request on this custom draing code. So whoever wants to restore this code
> should also be ready to look into bug reports in this area.
> 
> Bug 566281 sounds like an issue with Windows SWT and not something we should
> try to overpaint in the CSS engine.

I did only have a quick look at the code, I did not notice it uses the custom painter in this version already. I agree with you that this is best made customizable in SWT.
Comment 14 Rolf Theunissen CLA 2021-06-09 02:43:02 EDT
(In reply to Wayne Baldwin from comment #12)
> There are actually TWO colors here, background and foreground, are both
> controlled by SWT? How can so many other colors be customized but NOT
> selected items?

It seems not to complex, but so far nobody was annoyed enough to be willing to pay to implement this (either in time or money). For me, I don't have the time to fix it so I just stick with the light theme, such that I am not annoyed.

Given that this is an open source project, the DevStyle people are in control too, it is open for them to provide fixes to the issues they encounter.
Comment 15 Wayne Baldwin CLA 2021-06-09 08:10:47 EDT
(In reply to Rolf Theunissen from comment #14)
> (In reply to Wayne Baldwin from comment #12)
> > There are actually TWO colors here, background and foreground, are both
> > controlled by SWT? How can so many other colors be customized but NOT
> > selected items?
> 
> It seems not to complex, but so far nobody was annoyed enough to be willing
> to pay to implement this (either in time or money). For me, I don't have the
> time to fix it so I just stick with the light theme, such that I am not
> annoyed.
> 
> Given that this is an open source project, the DevStyle people are in
> control too, it is open for them to provide fixes to the issues they
> encounter.

The DevStyle people said they have no control over these colors. Are you saying that the SWT is open source? Or is the "CSS Engine" what you are referring to that they could modify like the Bugs above attempted to do but failed?
Comment 16 Lars Vogel CLA 2021-06-09 08:49:24 EDT
(In reply to Wayne Baldwin from comment #15)

> The DevStyle people said they have no control over these colors. Are you
> saying that the SWT is open source? Or is the "CSS Engine" what you are
> referring to that they could modify like the Bugs above attempted to do but
> failed?

All components of Eclipse including SWT are open source. Any interested party could contribute to it so if the DevStyle people want they could fix this at its source. See https://www.vogella.com/tutorials/EclipsePlatformDevelopment/article.html for a description for the process and examples how to do this.
Comment 17 Todd E. Williams CLA 2021-06-09 11:20:53 EDT
I'll comment on behalf of Genuitec since our Darkest Dark product has been referenced in the bug report.

I'll first thank the Eclipse committers for looking at the issue and verifying what we found in our internal review.

Rolf and Lars are correct that Genuitec could dedicate resources to resolve this issue. However, in our testing we've found that it only affects Darkest Dark on Windows 7, and not the later Windows releases. 

Since Windows 7 is end-of-life [1] and even Microsoft has stopped supporting it, I regret that we won't be allocating any resources to resolve this issue.

If the SWT team also decides to leave it unaddressed, I'll suggest that this issue be closed as WONT-FIX due to affecting only an EOL platform.

[1] https://www.microsoft.com/en-US/windows/windows-7-end-of-life-support-information
Comment 18 Wayne Baldwin CLA 2021-06-09 11:44:53 EDT
Windows 7 may be EoL but I continue to get Malicious Software Removal Tool updates and it's a very stable platform to work with literally my work environment. Upgrading my VM to Win 10 may break other tools I rely on not to mention the non stop updates Win 10 forces down your throat at least every five or six weeks which could also break things.

Just the slightest change to either background or foreground color would make this much more useable, is there no easy way to tweak on or the other?
Comment 19 Wayne Baldwin CLA 2021-06-09 12:14:28 EDT
Created attachment 286552 [details]
Debug View highlights fine

So why does the Debug view NOT have the same problem? (see new attachment).
Comment 20 Niraj Modi CLA 2022-01-18 06:43:51 EST
Dark-theme issues seen in attachments(attachment 286536 [details], attachment 286543 [details], attachment 286552 [details]) are not seen any more.
IMO, bug 567616 took care of these(may be some other bug as well which I don't recall currently)

*** This bug has been marked as a duplicate of bug 567616 ***
Comment 21 Wayne Baldwin CLA 2022-01-18 08:37:02 EST
This bug is still present so I am not sure why you are closing this. The duplicate is also closed but the problem has not been resolved for me anyway. Installing Win 10 is not an option.
Comment 22 Todd E. Williams CLA 2022-01-18 09:13:04 EST
(In reply to Wayne Baldwin from comment #21)
> This bug is still present so I am not sure why you are closing this. The
> duplicate is also closed but the problem has not been resolved for me
> anyway. Installing Win 10 is not an option.

Wayne, this bug is being closed because it won't be fixed by the Eclipse committers because Windows 7 is EOL.

I addressed Genuitec's similar stance on the issue in Comment #18.
Comment 23 Todd E. Williams CLA 2022-01-18 09:15:19 EST
I'll suggest that the Eclipse SWT team change the resolution of this bug from CLOSED DUPLICATE to WONTFIX to avoid further confusion as this is not the same bug #567616 since it only is being reported for Windows 7.
Comment 24 Niraj Modi CLA 2022-01-18 09:16:54 EST
(In reply to Todd E. Williams from comment #23)
> I'll suggest that the Eclipse SWT team change the resolution of this bug
> from CLOSED DUPLICATE to WONTFIX to avoid further confusion as this is not
> the same bug #567616 since it only is being reported for Windows 7.

Done.