Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 416218 - [CSS] Contribute dark theme to Eclipse
Summary: [CSS] Contribute dark theme to Eclipse
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.4   Edit
Hardware: PC All
: P3 enhancement with 1 vote (vote)
Target Milestone: 4.4 M6   Edit
Assignee: Lars Vogel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 385003 427127
  Show dependency tree
 
Reported: 2013-08-30 04:55 EDT by Lars Vogel CLA
Modified: 2014-04-08 08:00 EDT (History)
8 users (show)

See Also:


Attachments
image that demonstrates icon translucency problems (105.93 KB, image/png)
2013-09-01 22:12 EDT, Tony McCrary CLA
no flags Details
DarkTheme (64.45 KB, image/png)
2013-11-13 16:35 EST, Lars Vogel CLA
no flags Details
Screenshot (217.03 KB, image/png)
2014-01-31 10:12 EST, Lars Vogel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2013-08-30 04:55:56 EDT
As discussed on the e4 mailing list, it think the Eclipse platform should offer an dark theme. The purpose of this theme would be:

1.) Attract users which prefer a dark styled IDE (IntellJ, sublime users seem to prefer that)
2.) Show limitation in the current CSS styling (Button, table headers, etc.) and start working on them.

My proposal would be to contribute this via a separate plug-in. 

If the platform team agrees to this approach, please let me know how the plug-in should be named (Bundle-Symbolic-Name). I will try to coordinate with the existing dark style plug-in providers and to contribute an initial styling.
Comment 1 Tony McCrary CLA 2013-09-01 22:12:18 EDT
Created attachment 235066 [details]
image that demonstrates icon translucency problems

I don't own the rights to this image, it's just for demonstrating Eclipse's issue with platform icons on dark backgrounds.
Comment 2 Tony McCrary CLA 2013-09-01 22:13:11 EDT
Aside from CSS changes, most of the current workbench icons look bad on a dark background. There are many gifs which don't support true translucency like PNG and other formats. I've attached a random example from Google image search that demonstrates this (check the edges of the run/play icon).

We are working on replacement icons for similar reasons, which are available here:
https://code.google.com/p/eclipse-svg-icons/
Comment 3 Lars Vogel CLA 2013-10-07 07:10:46 EDT
I asked Andrea Guarinoni who created https://github.com/guari/eclipse-ui-theme if this theme could be contributed to the Eclipse platform. Via email he confirmed that we can use his theme as basis. I ask him to confirm this in this bug report.
Comment 4 Andrea Guarinoni CLA 2013-10-07 11:55:31 EDT
Hi, as said to Lars, you can use it as you like if it might be of some help.
Comment 5 Lars Vogel CLA 2013-11-13 16:35:06 EST
Created attachment 237451 [details]
DarkTheme

Sample screenshot attached with the dark theme contributed by Andrea.
Comment 6 Lars Vogel CLA 2013-11-13 16:37:11 EST
I plan to contribute the new theme in its own plug-in. Current name is "org.eclipse.ui.theme.dark". If no one disagrees with that in the next days, I talk that as agreement.
Comment 7 Paul Webster CLA 2013-11-14 10:39:01 EST
(In reply to Lars Vogel from comment #6)
> I plan to contribute the new theme in its own plug-in. Current name is
> "org.eclipse.ui.theme.dark". If no one disagrees with that in the next days,
> I talk that as agreement.

Why not put it in org.eclipse.platform like the other themes?  Or arguably, do we need to move the other themes out into their own plugin (not in this bug, but another)?

PW
Comment 8 Andrea Guarinoni CLA 2013-11-14 11:10:58 EST
Is there a way to automatically associate an .epf configuration for syntax highlighting with a stylesheet/plugin? So we can have it applied as default when the dark theme is selected.
Comment 9 Paul Webster CLA 2013-11-14 11:36:12 EST
(In reply to Andrea Guarinoni from comment #8)
> Is there a way to automatically associate an .epf configuration for syntax
> highlighting with a stylesheet/plugin? So we can have it applied as default
> when the dark theme is selected.

There isn't at the moment.  But in Luna there is work to allow CSS to reference existing Color and Font definitions.  It might make sense to provide a way to look a CSS theme with a more traditional Color and Font theme.

PW
Comment 10 Lars Vogel CLA 2013-11-14 12:39:18 EST
(In reply to Paul Webster from comment #7)
> (In reply to Lars Vogel from comment #6)
> > I plan to contribute the new theme in its own plug-in. Current name is
> > "org.eclipse.ui.theme.dark". If no one disagrees with that in the next days,
> > I talk that as agreement.
> 
> Why not put it in org.eclipse.platform like the other themes?  Or arguably,
> do we need to move the other themes out into their own plugin (not in this
> bug, but another)?

I think themes relate to UI and should be part of the platform UI project. And I would like to make the themes consumable for RCP applications, this mean that a theme like dark theme should live in its own plug-in. See Bug 420836 for my suggestion to move the other CSS files also to their own plug-in in platform ui.
Comment 11 Lars Vogel CLA 2014-01-31 10:08:04 EST
I converted and uploaded the initial contribution from Andrea. My preference would be to get this in as is (if the plug-in structure is not obviously wrong) and start improving it. I hope for example that based on the CSS work from Daniel Rolka we might be able to simplify the styling.
Comment 12 Lars Vogel CLA 2014-01-31 10:11:00 EST
https://git.eclipse.org/r/#/c/21392/
Comment 13 Lars Vogel CLA 2014-01-31 10:12:15 EST
Created attachment 239527 [details]
Screenshot
Comment 14 Lars Vogel CLA 2014-02-04 11:57:26 EST
First contributions merged with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=dd57ff612541e64ec8b14945c41cf259485c8a1c

Looking at the existing plug-ins I suggest to name it either org.eclipse.e4.ui.themes or org.eclipse.ui.themes. 

https://git.eclipse.org/r/21521
Comment 15 Lars Vogel CLA 2014-02-04 12:00:41 EST
Change the name to org.eclipse.e4.ui.themes with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=e6c88938560769a2b7d607e9d4eb6917601b01f3

IIRC someone had concerns about org.eclipse.ui.themes, which I would prefer. I leave this bug open for a while to allow for feedback on the bundle symbolic name.

Thanks Andrea for this great contribution!!!!
Comment 16 Lars Vogel CLA 2014-02-04 14:44:13 EST
Paul Webster suggested via IRC to use "org.eclipse.ui.themes". 

Adjusted with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=687d6d1b32cd040bf431d33bf8397c91655973f0
Comment 17 Dani Megert CLA 2014-04-08 08:00:29 EDT
This change was not completely correct. The bundle now appears like this in the About dialog:
providerName   pluginName
You could have easily seen/fix this by looking at the warnings.

Fixed with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=071b6be09f82cfdc8d46d55bc1d003b983874f26