Bug 472707

Summary: Allow user to disable CSS styling
Product: [Eclipse Project] Platform Reporter: Nobody - feel free to take it <nobody>
Component: UIAssignee: Nobody - feel free to take it <nobody>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: akurtakov, arunkumar.thondapu, daniel_megert, Lars.Vogel, loskutov, peter, psuzzi
Version: 4.6Keywords: noteworthy
Target Milestone: 4.6 M1   
Hardware: PC   
OS: Linux   
See Also: https://git.eclipse.org/r/51968
https://git.eclipse.org/r/52338
https://git.eclipse.org/r/52488
https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=500e8529b6f541fae6caf3549a95da9e67cecf7f
https://git.eclipse.org/r/52883
https://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=8db3440aaba92b77bbcc70760e1236191d18fac8
https://bugs.eclipse.org/bugs/show_bug.cgi?id=516998
Whiteboard:
Bug Depends on: 472761    
Bug Blocks: 474237, 474320, 474439, 474444, 497634, 516998, 519469    
Attachments:
Description Flags
Proposed look none

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.