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

Bug 345597

Summary: [Theming] Background color is not applied
Product: [RT] RAP Reporter: Oliver B. <oliver>
Component: RWTAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P2 CC: steffen
Version: 1.4   
Target Milestone: 1.4 RC1   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Example
none
Proposed patch ivan: review?

Description Oliver B. CLA 2011-05-12 09:24:11 EDT
Created attachment 195495 [details]
Example

The background color of Text/Combo is not always applied. The standard css theme value will be used initialy. But when a color is set manually with setBackground the new color will not be applied. The same happens with css-states like read-only.

This was tested using the following themes:
own custom theme, rap-default and fancy

The old classic theme does not have this problem.

A small sample dialog is attached.
The background of CCombo, DateTime, Label is working. But not for Text/Combo.

Tested wird RAP 1.4M7 and HEAD (RWT-standalone).
Last tested working version was RAP 1.4M5.
Comment 1 Ivan Furnadjiev CLA 2011-05-13 04:10:24 EDT
The reason for this issue is the new default background gradient set on Combo/Text by the default theme. As a temporary workaround, reset the Combo/Text CSS background-image property to none in your custom CSS.
Comment 2 Oliver B. CLA 2011-05-13 04:18:17 EDT
The workaround with the css backround-image worked.
Comment 3 Ivan Furnadjiev CLA 2011-05-16 04:17:37 EDT
Because of the default background gradient, the background mode does not work for Text/Combo as well. Reproducible with Controls Demo -> Composite tab, change "background mode" radio buttons.
Comment 4 Ivan Furnadjiev CLA 2011-05-16 05:13:22 EDT
Created attachment 195700 [details]
Proposed patch

This patch remove/reset background gradient from the server-side if background color have to be set.
Comment 5 Ralf Sternberg CLA 2011-05-16 15:48:05 EDT
The patch looks good to me. Applied to CVS HEAD.