Bug 559312 - [Dark theme] Tab color settings are being ignored
Summary: [Dark theme] Tab color settings are being ignored
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.14   Edit
Hardware: PC Linux
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2020-01-18 02:59 EST by David Vavra CLA
Modified: 2020-09-22 14:28 EDT (History)
9 users (show)

See Also:


Attachments
Tabs in 2019-09 (14.05 KB, image/png)
2020-01-18 02:59 EST, David Vavra CLA
no flags Details
Tabs in 2019-12 (18.10 KB, image/png)
2020-01-18 04:38 EST, David Vavra CLA
davavra: review?
Details
Tabs 2019-12 after using ACTIVE_TAB_BG_END (15.34 KB, image/png)
2020-05-30 23:54 EDT, David Vavra CLA
no flags Details
Background after Apply (8.60 KB, image/png)
2020-09-22 14:13 EDT, David Vavra CLA
no flags Details
AfterApplyAndClose (8.60 KB, image/png)
2020-09-22 14:15 EDT, David Vavra CLA
no flags Details
AfterApplyAndClose (8.82 KB, image/png)
2020-09-22 14:18 EDT, David Vavra CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description David Vavra CLA 2020-01-18 02:59:13 EST
Created attachment 281536 [details]
Tabs in 2019-09

I've been unable to change the text editor background to black in 2019-12 (v4.14)
Also can't change color of Active/Inactive tabs in 2019-12 (v4.14)
They remain the default colors.
Works fine in  2019-09 (v4.13)

I've attached an image of the tabs in 2019-09

I think they are set in <proj>/.settings/org.eclipse.ui.workbench.prefs
and <proj>/.metadata/.plugins/org.eclipse.core.runtime/.settings/org.eclipse.ui.workbench.prefs

with the following settings which are identical in both 2019-12 and 2019-09:

7:ColorsAndFontsPreferencePage.selectedElement=Corg.eclipse.ui.workbench.INACTIVE_TAB_BG_START
18:de.loskutov.EclipseSkins.tab.color.active=30,144,255
19:de.loskutov.EclipseSkins.tab.color.notactive=173,216,230
57:org.eclipse.e4.ui.css.theme.e4_dark.org.eclipse.ui.workbench.ACTIVE_TAB_BG_START=78,154,6
58:org.eclipse.e4.ui.css.theme.e4_dark.org.eclipse.ui.workbench.INACTIVE_TAB_BG_START=52,101,164
80:org.eclipse.ui.workbench.ACTIVE_TAB_TEXT_COLOR=233,185,110

in <proj>/.metadata/.plugins/org.eclipse.core.runtime/.settings/org.eclipse.ui.editors.prefs
the background colors are set to:
1:AbstractTextEditor.Color.Background=0,0,0
5:AbstractTextEditor.Color.SelectionForeground=0,0,0

But the 2019-12 version also has 
AbstractTextEditor.Color.Background.SystemDefault=false
AbstractTextEditor.Color.SelectionForeground.SystemDefault=false


Have the color settings moved?
Does some other setting need to be set?
Is something overriding them?
Comment 1 David Vavra CLA 2020-01-18 04:38:01 EST
Created attachment 281537 [details]
Tabs in 2019-12
Comment 2 Andrey Loskutov CLA 2020-01-18 04:49:08 EST
Lars, I believe you have optimized Dark mode in 4.14. Is this related? I don't use it, so can't say what is expected.
Comment 3 Lars Vogel CLA 2020-01-18 05:07:06 EST
I plan to have a look during the 4.15 time frame.
Comment 4 Lars Vogel CLA 2020-04-15 04:57:44 EDT
Adding Pierre-Yves and Mike, which might be able to help here.
Comment 5 Pierre-Yves Bigourdan CLA 2020-04-15 14:03:30 EDT
Previously, the Dark theme had the following CSS for inactive tabs:
swt-selected-tab-fill: '#org-eclipse-ui-workbench-INACTIVE_TAB_BG_START' '#org-eclipse-ui-workbench-INACTIVE_TAB_BG_END' 100%;

And the following for active tabs:
swt-selected-tab-fill: '#org-eclipse-ui-workbench-ACTIVE_TAB_BG_START' '#org-eclipse-ui-workbench-ACTIVE_TAB_BG_END' 100%;

Nowadays, these are used instead:
swt-selected-tab-fill: '#org-eclipse-ui-workbench-ACTIVE_TAB_BG_END';
background-color: '#org-eclipse-ui-workbench-INACTIVE_TAB_BG_END'

My guess would be that this was changed to make colours in the Dark theme more consistent, but I haven't checked the entire commit history (it's quite dense and a lot has moved around/been renamed).


Do things work again if you replace START with END in your preferences?
57:org.eclipse.e4.ui.css.theme.e4_dark.org.eclipse.ui.workbench.ACTIVE_TAB_BG_END=78,154,6
58:org.eclipse.e4.ui.css.theme.e4_dark.org.eclipse.ui.workbench.INACTIVE_TAB_BG_END=52,101,164
Comment 6 Kalyan Prasad Tatavarthi CLA 2020-05-21 04:13:43 EDT
Moving to 4.16 RC1
Comment 7 David Vavra CLA 2020-05-30 23:54:50 EDT
Created attachment 283107 [details]
Tabs 2019-12 after using ACTIVE_TAB_BG_END
Comment 8 David Vavra CLA 2020-05-31 00:00:11 EDT
Sorry about the delay. Don't know how I missed the update.
---

Changing ACTIVE_TAB_BG_START to ACTIVE_TAB_BG_END does alter the tab color (see attachment)  but has the unexpected side-effect of outlining the tab area. Annoying but better than before.
Comment 9 David Vavra CLA 2020-09-22 14:13:50 EDT
Created attachment 284225 [details]
Background after Apply
Comment 10 David Vavra CLA 2020-09-22 14:15:29 EDT
Created attachment 284226 [details]
AfterApplyAndClose
Comment 11 David Vavra CLA 2020-09-22 14:18:08 EDT
Created attachment 284227 [details]
AfterApplyAndClose
Comment 12 David Vavra CLA 2020-09-22 14:21:57 EDT
Comment on attachment 284225 [details]
Background after Apply

In both v4.16.0 and v4.17.0, the background is changed after selecting and pushing "Apply" but reverts when "Apply and Close" is pushed.

FWIW: Also changed after I took a screenshot.
Comment 13 David Vavra CLA 2020-09-22 14:28:36 EDT
Just to clarify:
The line number background retains the background color selection but the text body reverts to that gray color. The "After Apply and Close" image tried setting the color to light blue.