Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 430370

Summary: [CSS] Provide CSS handler for Forms Section element
Product: [Eclipse Project] Platform Reporter: Lars Vogel <Lars.Vogel>
Component: UIAssignee: Simon Scholz <simon.scholz>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: daniel.rolka, daniel_megert, Lars.Vogel, mihael, peter, simon.scholz
Version: 4.4   
Target Milestone: 4.5 M5   
Hardware: PC   
OS: Linux   
Whiteboard:
Bug Depends on: 430205    
Bug Blocks:    
Attachments:
Description Flags
Screenshot
none
The border of the titblebar is white and its background is also dark(#6D5E5E)
none
Before
none
After none

Description Lars Vogel CLA 2014-03-14 05:59:01 EDT
See screenshot. Daniel, if you point me to the right place I can try to fix that.
Comment 1 Lars Vogel CLA 2014-03-14 05:59:49 EDT
Created attachment 240898 [details]
Screenshot
Comment 2 Lars Vogel CLA 2014-03-18 17:09:45 EDT
I think for this we require an additional plug-in to avoid a dependency conflict. I suggest to call it: 

org.eclipse.e4.ui.css.forms

I create a first version of it, but it would be great if Daniel good review that, as he has written the reset for the SWT CSS.
Comment 3 Lars Vogel CLA 2014-03-18 17:21:46 EDT
I try to solve this together with Bug 430205
Comment 4 Lars Vogel CLA 2014-03-18 17:23:32 EDT
https://git.eclipse.org/r/23565

To test this change switch between dark and light themes with a "Section" in an editor open.
Comment 5 Lars Vogel CLA 2014-04-28 12:19:17 EDT
Moving to 4.5 as of Pauls request, I plan to see if I still can get that fixed and applied for 4.4 as it is pretty annoying for the dark theme.
Comment 6 Lars Vogel CLA 2014-04-28 13:16:21 EDT
Moving back to RC1.
Comment 7 Lars Vogel CLA 2014-11-26 10:22:57 EST
Here is some related discussion in IRC: http://echelog.com/logs/browse/eclipse-e4/1395702000
Comment 8 Simon Scholz CLA 2014-11-26 12:21:18 EST
I just rebased the patch from Lars and added also the styling for the title bar border.

See: https://git.eclipse.org/r/#/c/23565/
Comment 9 Lars Vogel CLA 2014-11-26 12:41:00 EST
(In reply to Simon Scholz from comment #8)
> I just rebased the patch from Lars and added also the styling for the title
> bar border.
> 
> See: https://git.eclipse.org/r/#/c/23565/

Can you add a screenshot how the title bar looks like? I would prefer a solid color (start gradient == end gradient).
Comment 10 Simon Scholz CLA 2014-11-26 12:45:17 EST
Created attachment 248970 [details]
The border of the titblebar is white and its background is also dark(#6D5E5E)
Comment 11 Lars Vogel CLA 2014-11-26 14:31:47 EST
(In reply to Lars Vogel from comment #9)
> Can you add a screenshot how the title bar looks like? I would prefer a
> solid color (start gradient == end gradient).

Solid color is handled by Bug 430205.
Comment 12 Lars Vogel CLA 2014-12-15 17:19:18 EST
(In reply to Simon Scholz from comment #8)
> See: https://git.eclipse.org/r/#/c/23565/

This does not work for me, I attach a before switching to dark screenshot and a after switching to dark and back screenshot. I would expect that the forms UI looks the same after switching back from dark to GTK.
Comment 13 Lars Vogel CLA 2014-12-15 17:19:59 EST
Created attachment 249448 [details]
Before
Comment 14 Lars Vogel CLA 2014-12-15 17:20:22 EST
Created attachment 249449 [details]
After
Comment 15 Lars Vogel CLA 2015-01-08 04:12:30 EST
(In reply to Lars Vogel from comment #12)
> (In reply to Simon Scholz from comment #8)
> > See: https://git.eclipse.org/r/#/c/23565/
> 
After some rework from Simon, merged with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=9b58903d98a9c85379b836c057b94d514c238899
Comment 16 Dani Megert CLA 2015-01-09 06:12:16 EST
(In reply to Lars Vogel from comment #15)
> (In reply to Lars Vogel from comment #12)
> > (In reply to Simon Scholz from comment #8)
> > > See: https://git.eclipse.org/r/#/c/23565/
> > 
> After some rework from Simon, merged with
> http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=9b58903d98a9c85379b836c057b94d514c238899
> 

I don't think you really wanted to introduce two new API packages. I've converted them to internal and registered them in the manifest. I've also increased the bundle version as required by https://wiki.eclipse.org/Version_Numbering.

Fixed with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=e87165eb4754a41bfcf18cc674260f099c5daaa1
Comment 17 Lars Vogel CLA 2015-01-09 06:26:48 EST
Thanks Dani for watching out.
Comment 18 Lars Vogel CLA 2015-01-28 10:06:09 EST
Validated via the dark theme in 4.5.0.I20150126-2000. The new gradient has a drawing issue, I follow up on this via Bug 430205.