Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 319936 - Toolbar separators hardly visible with Windows XP blue theme
Summary: Toolbar separators hardly visible with Windows XP blue theme
Status: RESOLVED WORKSFORME
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Bogdan Gheorghe CLA
QA Contact: Susan McCourt CLA
URL:
Whiteboard: stalebug
Keywords:
Depends on:
Blocks: 293481
  Show dependency tree
 
Reported: 2010-07-14 22:52 EDT by Boris Bokowski CLA
Modified: 2019-06-05 07:37 EDT (History)
5 users (show)

See Also:


Attachments
Screenshot depicting the problem in question. (9.73 KB, image/png)
2010-07-15 07:16 EDT, Remy Suen CLA
no flags Details
screenshot on win7 - workbench using win xp blue stylesheet (27.96 KB, image/png)
2010-07-15 18:01 EDT, Susan McCourt CLA
no flags Details
blind patch (2.31 KB, patch)
2010-07-15 18:21 EDT, Susan McCourt CLA
no flags Details | Diff
CSS from org.eclipse.platform (959 bytes, text/plain)
2010-07-15 18:22 EDT, Susan McCourt CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Boris Bokowski CLA 2010-07-14 22:52:26 EDT
We should investigate if the toolbar separator can be styled (different color?) or if we need to adjust the toolbar background color. Alternatively, we could use an image instead of the OS-drawn separator.
Comment 1 Boris Bokowski CLA 2010-07-14 22:55:13 EDT
Oleg or Remy, could one of you please attach a screenshot of Eclipse 4.0 (blue theme) on Windows XP? Thanks!
Comment 2 Remy Suen CLA 2010-07-15 07:16:20 EDT
Created attachment 174384 [details]
Screenshot depicting the problem in question.

(In reply to comment #1)
> Oleg or Remy, could one of you please attach a screenshot of Eclipse 4.0 (blue
> theme) on Windows XP? Thanks!

See attached.
Comment 3 Remy Suen CLA 2010-07-15 07:16:34 EDT
Forgot to say that comment 2 is from I20100714-2048.
Comment 4 Susan McCourt CLA 2010-07-15 12:09:11 EDT
Linda caught this in a review last week (that I'm just catching up with) - she had flagged it as "separators are missing on XP" so she didn't realize it was a color problem.  The mockups used the coolbar separators and so they were visible.

I noticed in the mockups there is an (accidental) 3-d looking separator alongside the other coolbar separators.  With the lighter line next to the dark line, it's more visible.  Is there an OS-style we could change?
Comment 5 Boris Bokowski CLA 2010-07-15 12:27:30 EDT
(In reply to comment #4)
> I noticed in the mockups there is an (accidental) 3-d looking separator
> alongside the other coolbar separators.  With the lighter line next to the dark
> line, it's more visible.  Is there an OS-style we could change?

Was this a question for Bogdan?
Comment 6 Susan McCourt CLA 2010-07-15 18:01:12 EDT
Created attachment 174457 [details]
screenshot on win7 - workbench using win xp blue stylesheet

I'm assuming this is a platform-specific problem that depends on how the separator is rendered on the user's platform.  On Win7, a 3-D separator is used, so that even on the win xp blue theme, the separators are visible.  (They don't "pop," but they are acceptable I think).

So the question is how to style a ToolItem of style SWT.SEPARATOR??
Comment 7 Susan McCourt CLA 2010-07-15 18:21:17 EDT
Created attachment 174458 [details]
blind patch

attempt at a patch - adds a CSS ID to all of the tool item separators created by the SeparatorRenderer and ToolBarRenderer.  Sets a color for the separator in the win xp blue style sheet (to be attached separately).

I don't see any effect, but if win7 separators are always 3D, the color may simply be ignored.  Needs to be tried on win XP.
Comment 8 Susan McCourt CLA 2010-07-15 18:22:42 EDT
Created attachment 174459 [details]
CSS from org.eclipse.platform

CSS.

Also, do we know if WinXp Olive is hard to see separators on WinXP?
Comment 9 Susan McCourt CLA 2010-07-15 18:25:46 EDT
this patch is really me guessing what to do.
- I'm not sure if ToolItems work the same as do actual controls with respect to the CSS Engine.  This patch assumes they do.
- I'm not sure I've chosen a good color.  We could go either extra dark, or we could go white/very light blue.  Would have to try both and compare.  This patch does extra dark gray.
- I'm not sure the foreground color is the right color to set on a tool separator assuming the rest of my assumptions are correct.
Comment 10 Susan McCourt CLA 2010-08-16 17:55:38 EDT
my "guesswork" patch didn't work, the OS won't let you set color on a separator.

We had a brief chat with Linda and the direction going forward is to see if we can lighten up the toolbar gradient enough to see the separator.  We want to avoid drawing our own separators if possible.  Bogdan's going to experiment and get Linda's feedback.
Comment 11 Eclipse Genie CLA 2018-11-09 18:51:03 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 12 Lars Vogel CLA 2019-06-05 07:37:39 EDT
This is a mass change to close all e4 bugs marked with "stalebug" whiteboard.

If this bug is still valid, please reopen and remove the "stalebug" keyword.