This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 562227 - Remove themes for other platforms
Summary: Remove themes for other platforms
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.16   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: 4.16 M3   Edit
Assignee: Lars Vogel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 562223 (view as bug list)
Depends on: 527806
Blocks:
  Show dependency tree
 
Reported: 2020-04-16 11:39 EDT by Mike Marchand CLA
Modified: 2020-05-05 10:22 EDT (History)
6 users (show)

See Also:
daniel_megert: pmc_approved+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Marchand CLA 2020-04-16 11:39:44 EDT
In the preference dialog, under General > Appearance, we should hide themes for other platforms by default in the theme selection dropdown.

From Lars Vogel on Bug 562221:
"They were explicitly added in the past but I think we should add a checkbox: 'Show themes for other OSs' and show them only if selected by the user in the dialog.
Comment 1 Andrew Obuchowicz CLA 2020-04-16 11:44:10 EDT
Just to give a potential hint with solving this bug:
Here is a patch which detects if the user is on Linux (GTK) https://git.eclipse.org/r/#/c/154753/4/org.eclipse.ui.editors/src/org/eclipse/ui/internal/editors/text/TextEditorDefaultsPreferencePage.java
Comment 2 Mickael Istria CLA 2020-04-28 11:11:07 EDT
The reason is that some people did want the Linux theme on Windows.
What bug would hiding them solve? What added value would it offer.
IMO, it's not worth the effort as it increases complexity of the UI for not substantial benefit.
Comment 3 Lars Vogel CLA 2020-04-28 11:14:32 EDT
PMC decided that showing the OS specific themes is more confusing than helpful and that we shoud remove them. No preference is needed.
Comment 4 Lars Vogel CLA 2020-04-28 11:15:20 EDT
(In reply to Mickael Istria from comment #2)
> The reason is that some people did want the Linux theme on Windows.
> What bug would hiding them solve? What added value would it offer.
> IMO, it's not worth the effort as it increases complexity of the UI for not
> substantial benefit.

We should improve the Windows theme. Opened https://bugs.eclipse.org/bugs/show_bug.cgi?id=551462 for this.
Comment 5 Mickael Istria CLA 2020-04-28 11:19:55 EDT
(In reply to Lars Vogel from comment #3)
> PMC decided that showing the OS specific themes is more confusing than
> helpful and that we shoud remove them. No preference is needed.

I think PMC is making a mistake here and that decreases the value of the IDE. Offering multiple themes is a way to mitigate the "default theme sucks" issue that's quite frequent on Windows (more than with other OSs) by offering alternatives.

> We should improve the Windows theme. Opened https://bugs.eclipse.org/bugs/show_bug.cgi?id=551462 for this.

Sure, but in the meantime, it has been horrible for years, and no one has worked on some redesign. I don't get why that would suddenly change and why preventing from offering alternatives.
Note that the themes are *not* OS specific, they happen to "map" the usual default OS settings, they're... themes, not bound to OS, just like one can decide to use some Mac-inspired theme on Gnome.
Comment 6 Lars Vogel CLA 2020-04-28 11:24:21 EDT
(In reply to Mickael Istria from comment #5)

> Sure, but in the meantime, it has been horrible for years, and no one has
> worked on some redesign. I don't get why that would suddenly change 
 
Because I started using Windows for learning video recording. ;-) I already pushed a Gerrit patch which makes Windows toolbar styling theme IMHO much better (and similar to the Linux theming).
Comment 8 Eclipse Genie CLA 2020-04-28 11:40:07 EDT
New Gerrit change created: https://git.eclipse.org/r/161673
Comment 9 Lars Vogel CLA 2020-04-28 11:41:38 EDT
*** Bug 562223 has been marked as a duplicate of this bug. ***
Comment 10 Alexander Kurtakov CLA 2020-04-28 11:47:29 EDT
(In reply to Mickael Istria from comment #5)
> (In reply to Lars Vogel from comment #3)
> > PMC decided that showing the OS specific themes is more confusing than
> > helpful and that we shoud remove them. No preference is needed.
> 
> I think PMC is making a mistake here and that decreases the value of the
> IDE. Offering multiple themes is a way to mitigate the "default theme sucks"
> issue that's quite frequent on Windows (more than with other OSs) by
> offering alternatives.

Offering more alternatives when we fail to maintain and improve even the default ones is not helping streamline the efforts. I can see the point in having this as hidden gem (hidden behind system property? for people working on themes so comparisons are easy and etc.). Offering more and more themes is not the way to go - this started from proposal to have additional themes/options for rounded and square ctabfolder rendering. That would have doubled the current themes and going that path we would have to add search field in the combo soon enough.

> 
> > We should improve the Windows theme. Opened https://bugs.eclipse.org/bugs/show_bug.cgi?id=551462 for this.
> 
> Sure, but in the meantime, it has been horrible for years, and no one has
> worked on some redesign. I don't get why that would suddenly change and why
> preventing from offering alternatives.

We see a change right now so let's see how long Windows theme improvements will continue.

> Note that the themes are *not* OS specific, they happen to "map" the usual
> default OS settings, they're... themes, not bound to OS, just like one can
> decide to use some Mac-inspired theme on Gnome.

This is almost true - but they also contain specifics which can make the theme look really inconsistent. E.g. the gtk theme has definitions about themes selected/unselected/etc. tab colors as gtk doesn't manage these but cocoa does as they are not defined there. This could lead to inconsistent selection colors on cocoa if gtk theme used. And probably there are more like that. So while in general themes are mapping colors only - they are not tested for other platforms so we shouldn't offer them to users by default.
Comment 11 Mike Marchand CLA 2020-04-28 11:48:15 EDT
In our Eclipse based product, we have added our own dark theme and the list of themes has grown substantially and confuses users.  You can't say themes have nothing to do with OS if we have 3 dark themes that are tuned to each OS and have tags like (Windows) and (Linux)...

To be honest.  I would prefer a mechanism to completely remove all platform contributed themes so that our product could offer only it's own themes and those downloaded by users.
Comment 12 Matthias Becker CLA 2020-05-05 10:20:09 EDT
So this bug is basically a revert of https://bugs.eclipse.org/bugs/show_bug.cgi?id=527806 ? 
I am okay with that...
Comment 13 Lars Vogel CLA 2020-05-05 10:21:53 EDT
Thanks Lakshmi and Matthias for checking.