This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 371785 - Remove "Default" theme because it is broken (and redundant as every platform has its default theme)
Summary: Remove "Default" theme because it is broken (and redundant as every platform ...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.2   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 4.4 M4   Edit
Assignee: Lars Vogel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 421387 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-02-16 11:19 EST by John Arthorne CLA
Modified: 2013-12-12 06:32 EST (History)
9 users (show)

See Also:


Attachments
Screen shot (56.36 KB, image/png)
2012-02-16 11:19 EST, John Arthorne CLA
no flags Details
Classic theme on Linux (64.61 KB, image/png)
2012-12-22 17:55 EST, Andrey Loskutov CLA
no flags Details
Default theme on Linux (70.25 KB, image/png)
2012-12-22 17:55 EST, Andrey Loskutov CLA
no flags Details
Gnome theme on Linux (77.25 KB, image/png)
2012-12-22 17:56 EST, Andrey Loskutov CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description John Arthorne CLA 2012-02-16 11:19:10 EST
I accidentally got myself into the "Default" theme on the Appearance preference page. On the next restart I discovered that it looks hideous. There are extra black lines all over the place, and some huge margin padding between toolbar and view/editor panes. We should either make this pretty or remove it, and in its current state it shouldn't be called "Default".
Comment 1 John Arthorne CLA 2012-02-16 11:19:41 EST
Created attachment 211120 [details]
Screen shot
Comment 2 Eric Moffatt CLA 2012-04-04 14:05:41 EDT
+1 from me, we should either remove it or fix it (and removing it is easier...;-).

BTW I also get a couple of errors in the console...

java.io.FileNotFoundException: .\winXPBlue.PNG (The system cannot find the file specified)

not sure what's up with that...
Comment 3 Dani Megert CLA 2012-04-05 06:59:38 EDT
If we remove it, we need to make sure that if people start an old (including 3.x) workspace with default appearance get redirected to some existing theme.
Comment 4 Mike Wilson CLA 2012-04-17 15:36:16 EDT
> I accidentally got myself into the "Default" theme on the Appearance preference page.
>
How hard is it to make that happen? Does switching to other themes trash the appearance? Should we be removing the whole theming preference page? (Actually, I thought we already had)
Comment 5 Eric Moffatt CLA 2012-05-07 10:07:18 EDT
*** Bug 349867 has been marked as a duplicate of this bug. ***
Comment 6 Eric Moffatt CLA 2012-05-07 10:15:43 EDT
Moving to RC1...
Comment 7 Andrey Loskutov CLA 2012-12-22 17:55:25 EST
Created attachment 225018 [details]
Classic theme on Linux

Honestly there isn't much choices on Linux: "Default" is utterly broken as described by John, "Gnome" isn't much better in terms of look and feel (the only vertical padding is not so huge and blue color is replaced to gray), and the only usable "Classic" has some cheese and pixel artifacts and ... mru editor tabs order. I will attach pictures of each theme on my Kubuntu 12.10.

So from professional point of view only "Classic" looks nice (beside few pixel artifacts).

Regarding removal of themes selection - why? Shouldn't at least some 3rd party developers be able to provide new (usable) themes?
Comment 8 Andrey Loskutov CLA 2012-12-22 17:55:44 EST
Created attachment 225019 [details]
Default theme on Linux
Comment 9 Andrey Loskutov CLA 2012-12-22 17:56:02 EST
Created attachment 225020 [details]
Gnome theme on Linux
Comment 10 Dani Megert CLA 2013-01-04 04:19:45 EST
I would defer this to 4.3.
Comment 11 John Arthorne CLA 2013-01-04 09:23:00 EST
Yes, we are in the 4.2.2 end-game now.
Comment 12 Dani Megert CLA 2013-05-29 03:04:36 EDT
Too late for Kepler.
Comment 13 Dani Megert CLA 2013-11-01 11:27:26 EDT
(In reply to Dani Megert from comment #3)
> If we remove it, we need to make sure that if people start an old (including
> 3.x) workspace with default appearance get redirected to some existing theme.

I set the theme to 'Default', then deleted it and restarted. The workspace simply switched to 'Windows 7' without any complaint. I also looked for "default.noos" in the code base and found no reference.

Paul, Erich, any idea whether this could be used elsewhere?
Comment 14 Lars Vogel CLA 2013-11-02 19:30:37 EDT
> I set the theme to 'Default', then deleted it and restarted. The workspace
> simply switched to 'Windows 7' without any complaint. I also looked for
> "default.noos" in the code base and found no reference.

Same here, removing the default theme is handled well. And it is not referred in the eclipse.platform or eclipse.platform.ui repository. 
[Search via: grep -r "e4_default.css" .]

I think we are save to remove the theme.
Comment 15 Lars Vogel CLA 2013-11-02 19:33:06 EDT
https://git.eclipse.org/r/#/c/18011/1
Comment 17 Dani Megert CLA 2013-11-04 06:51:48 EST
(In reply to Lars Vogel from comment #16)
> Thanks for merging Dani. Fixed with
> https://git.eclipse.org/c/platform/eclipse.platform.git/commit/
> ?id=c61f9ab80a8154f39c4132065ca5db0486250155

np, though I already had the change in my workspace ;-).

I removed the no longer used NLS key from 'plugin.properties' with http://git.eclipse.org/c/platform/eclipse.platform.git/commit/?id=7e778643de3f337f796617b5239cf994e3aa5754
Comment 18 Dani Megert CLA 2013-11-11 04:25:50 EST
*** Bug 421387 has been marked as a duplicate of this bug. ***
Comment 19 Lars Vogel CLA 2013-12-12 06:32:17 EST
Verified in Build id: I20131211-2000 #YEAH!