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

Bug 329348

Summary:

[Combo][Text][Spinner] Make selection background/foreground themeable

Product:

[RT] RAP

Reporter:

Wen Ai <wen.ai>

Component:

RWT

Assignee:

Project Inbox <rap-inbox>

Status:

NEW ---

QA Contact:

Severity:

enhancement

Priority:

CC:

rsternberg

Version:

unspecified

Target Milestone:

---

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
screenshot for the combos	none

Description Wen Ai

2010-11-03 10:11:34 EDT

Build Identifier: 

Do we have a chance to change the color for the selected textfield in a combo?  

This problem can be observed in the Controls-Demo of RAP 1.3.0.2010100615-1704. (Please see the screenshot.)

Many thanks!

Reproducible: Always

Steps to Reproduce:
1. start the controls demo with branding "tea"
2. open the tab "combo" and fill all of the three combos on the left side 
3. click on the first combo and click the tab-key
4. the textfield for the next combo would be selected in the color #0A246A (could wir change this color?)

Comment 1 Wen Ai

2010-11-03 10:12:20 EDT

Created attachment 182299 [details]
screenshot for the combos

Comment 2 Ivan Furnadjiev

2010-11-03 10:49:45 EDT

Currently the selection colors (background/foreground) are controlled by the OS theme. In Windows XP it is dark blue (like in your screenshot), in Windows 7, Ubuntu 10.04 it is light blue. Maybe we could support styling of this colors via CSS. Some initial information here: http://www.quirksmode.org/css/selection.html

Comment 3 Ivan Furnadjiev

2010-11-03 11:55:28 EDT

Here is another link for cross browser support of ::selection pseudo-selector
http://reference.sitepoint.com/css/pseudoelement-selection
Seems like IE does not support it at all and Firefox uses ::-moz-selection.

Comment 4 Wen Ai

2010-11-04 04:32:54 EDT

It would be great, if rap could support this via CSS :-)

Comment 5 Ivan Furnadjiev

2010-11-04 05:01:47 EDT

Changed the title accordingly.

Comment 6 Ivan Furnadjiev

2010-11-04 05:19:47 EDT

And one more link - https://developer.mozilla.org/En/CSS/::selection
IE9 will support the	::selection pseudo-selector. There is no way to fix this issue for IE < IE9.

Comment 7 Ralf Sternberg

2010-11-04 08:04:36 EDT

Great news that it works at least in some browsers. Could it be a problem if we introduce a :selected state for Text-like widgets that works only in newer browsers? We'd have to make this clear in the theming documentation.

Missing support in IE would not render the application useless, worst thing that can happen that the selected text looks slightly different in some browsers (which it does already, depending on the OS theming). From my POV, this would be fine.