Bug 472707 - Allow user to disable CSS styling
Summary: Allow user to disable CSS styling
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.6   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.6 M1   Edit
Assignee: Nobody - feel free to take it CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on: 472761
Blocks: 474237 474320 474439 474444 497634 516998 519469
  Show dependency tree
 
Reported: 2015-07-15 06:03 EDT by Nobody - feel free to take it CLA
Modified: 2017-07-10 11:44 EDT (History)
7 users (show)

See Also:


Attachments
Proposed look (138.63 KB, image/png)
2015-07-15 06:03 EDT, Nobody - feel free to take it CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nobody - feel free to take it CLA 2015-07-15 06:03:52 EDT
Created attachment 255206 [details]
Proposed look

I think it would be nice for people to have the option of having Eclipse look just like any other GTK application.

It's relatively small effort on our side to offer this to users. It would consist of only extremely minor adjustments and look like the attachment. I can imagine lots of people wanting a back to basic looks (no round edges etc. etc.).
Comment 1 Alexander Kurtakov CLA 2015-07-15 06:14:25 EDT
I would also expect noticable speed difference as this would reduce the calls significantly as eclipse is not going to override every color/egde/etc. independently via JNI calls.
Comment 2 Eclipse Genie CLA 2015-07-15 06:22:02 EDT
New Gerrit change created: https://git.eclipse.org/r/51968
Comment 3 Lars Vogel CLA 2015-07-15 06:31:59 EDT
I think such a theme should be available for all platform not only for GTK.

The following command line option does this already and removes completely the CSS overhead.

-cssTheme none

I think the right solution is allow styling disablement via a preference setting, this will result in native rendering and a faster IDE. I adjusted the title accordingly, I hope you guys agree.
Comment 4 Andrey Loskutov CLA 2015-07-15 06:52:04 EDT
(In reply to Lars Vogel from comment #3)
> I think such a theme should be available for all platform not only for GTK.
> 
> The following command line option does this already and removes completely
> the CSS overhead.
> 
> -cssTheme none
> 
> I think the right solution is allow styling disablement via a preference
> setting, this will result in native rendering and a faster IDE. I adjusted
> the title accordingly, I hope you guys agree.

Yep.
Comment 5 Nobody - feel free to take it CLA 2015-07-15 07:07:28 EDT
(In reply to Lars Vogel from comment #3)
> I think such a theme should be available for all platform not only for GTK.
> 
> The following command line option does this already and removes completely
> the CSS overhead.
> 
> -cssTheme none
> 
> I think the right solution is allow styling disablement via a preference
> setting, this will result in native rendering and a faster IDE. I adjusted
> the title accordingly, I hope you guys agree.

The issue with that is that not having :

MToolControl.Draggable {
	handle-image:  url(./dragHandle.png);
} 

completely disables the drag and drop.
Comment 6 Andrey Loskutov CLA 2015-07-15 07:26:23 EDT
(In reply to Sopot Cela from comment #5)
> The issue with that is that not having :
> 
> MToolControl.Draggable {
> 	handle-image:  url(./dragHandle.png);
> } 
> 
> completely disables the drag and drop.

Sopot, I'm working since half a year with -cssTheme none and did not noticed that (and I often use drag and drop with git history view). How do you see that this is not working, what specific issue do you have? I'm on Linux GTK2/Fedora 21 if this is important.
Comment 7 Nobody - feel free to take it CLA 2015-07-15 07:31:33 EDT
I should have been specific that it disables toolbar drag and drop. Can you confirm that you see the drag handle (the small vertical dashed line) in the toolbars (or toolcontrols such as the perspective switcher) ?
Comment 8 Andrey Loskutov CLA 2015-07-15 14:12:10 EDT
(In reply to Sopot Cela from comment #7)
> I should have been specific that it disables toolbar drag and drop. Can you
> confirm that you see the drag handle (the small vertical dashed line) in the
> toolbars (or toolcontrols such as the perspective switcher) ?

A-ha. Never noticed this until now, so I've opened bug 472761.
Comment 9 Eclipse Genie CLA 2015-07-22 07:13:22 EDT
New Gerrit change created: https://git.eclipse.org/r/52338
Comment 10 Eclipse Genie CLA 2015-07-24 08:19:54 EDT
New Gerrit change created: https://git.eclipse.org/r/52488
Comment 12 Nobody - feel free to take it CLA 2015-07-24 09:54:37 EDT
(In reply to Eclipse Genie from comment #11)
> Gerrit change https://git.eclipse.org/r/52488 was merged to [master].
> Commit:
> http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/
> ?id=500e8529b6f541fae6caf3549a95da9e67cecf7f

Fix merged. Will close this after I add the N&N change.
Comment 13 Lars Vogel CLA 2015-07-30 12:03:12 EDT
(In reply to Sopot Cela from comment #12)
> Fix merged. Will close this after I add the N&N change.

Ping.
Comment 14 Nobody - feel free to take it CLA 2015-07-30 12:09:14 EDT
(In reply to Lars Vogel from comment #13)
> (In reply to Sopot Cela from comment #12)
> > Fix merged. Will close this after I add the N&N change.
> 
> Ping.

Will do this by today for sure.
Comment 15 Eclipse Genie CLA 2015-07-30 12:43:53 EDT
New Gerrit change created: https://git.eclipse.org/r/52883
Comment 17 Nobody - feel free to take it CLA 2015-07-30 12:45:13 EDT
N&N is in.
Comment 18 Dani Megert CLA 2015-08-05 08:22:10 EDT
There's screen cheese with disabled CSS, see bug 474320.
Comment 19 Andrey Loskutov CLA 2015-08-05 08:30:18 EDT
(In reply to Dani Megert from comment #18)
> There's screen cheese with disabled CSS, see bug 474320.

AFAIK (I'm running without CSS long time now) this is exact only those few lines around toolbar/editor tab area, so not that serious as it sounds.
Comment 20 Dani Megert CLA 2015-08-05 08:35:51 EDT
(In reply to Andrey Loskutov from comment #19)
> (In reply to Dani Megert from comment #18)
> > There's screen cheese with disabled CSS, see bug 474320.
> 
> AFAIK (I'm running without CSS long time now) this is exact only those few
> lines around toolbar/editor tab area, so not that serious as it sounds.

Right, not serious, but ugly ;-).
Comment 21 Dani Megert CLA 2015-08-05 08:37:44 EDT
(In reply to Dani Megert from comment #20)
> (In reply to Andrey Loskutov from comment #19)
> > (In reply to Dani Megert from comment #18)
> > > There's screen cheese with disabled CSS, see bug 474320.
> > 
> > AFAIK (I'm running without CSS long time now) this is exact only those few
> > lines around toolbar/editor tab area, so not that serious as it sounds.
> 
> Right, not serious, but ugly ;-).

And now more widely visible, since the user can disable it via UI and since this will be mentioned in the M1 N&N.
Comment 22 Lars Vogel CLA 2017-05-19 12:03:11 EDT
This fix causes a regression of e4 related styling in e4 RCP applications. See Bug 516998.