1137 - Selection is wrong color

Status: VERIFIED FIXED

(Core :: DOM: Selection, defect, P4)

Description

[Angus Davis]

The selection should be the default selection color for the user's environment
(the OS tracks this somewhere), not black.

When I make a selection, the background selection color flickers.

Comment 1

[Greg Kostello]

Akkana -- now that you are owner of selection, I've assigned this bug to you.

Comment 2

[Akkana Peck]

Accepting bug ... we'll probably want to add a call at the xpfe or gfx level to
choose the selection color, potentially different for every platform; then just
call that routine from the xp selection code.

Comment 3

[Angus Davis]

Note that in CSS3, it's quite possible there will be a :selected pseudo class.
The default selection background color would be exposed as a valid CSS color
value much like the existing system color values in CSS2. The author would also
be able to specifiy (letter|word|line)-at-a-time selection using CSS.

Adding peterl to list since he's the CSS expert. Note: interested folks should
see the Microsoft User Interface extensions proposed to CSS on the W3C's site
(http://www.w3.org/Style/Group/) - you'll need a W3 user/pass to access it. It
contains syntax examples for some of this stuff.

Comment 4

[leger]

Setting all current Open/Normal to M4.

Comment 5

[Paul MacQuiddy]

per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 6

[Greg Kostello]

Change Milestone to M7

Comment 7

[Akkana Peck]

Passing this along to peterl.  We need someone knowledgeable about CSS to tell
us what we should be doing with selection color.

Comment 8

[leger]

Per a request from Selection and Search component eng (mjudge) and qa (elig),
moving all "Selection and Search" bugs to new "Selection" component.  Original
"Selection and Search" component will be retired.

Comment 9

[Eli Goldberg]

[QA Assigning to self.]

Comment 10

[Peter Linss]

Pushing off non-beta 1 issues

Comment 11

[Hixie (not reading bugmail)]

See the recently released CSS3 UI draft:
   http://www.w3.org/TR/1999/WD-css3-userint-19990916

See also bug 15282, which gives a suggested way of fixing this.

Basically, you want to be doing this:
   :selection { background-color: Highlight; color: HighlightText; }
...and use the CSS3 functionality mentioned above.

Comment 12

[Hixie (not reading bugmail)]

*** Bug 7086 has been marked as a duplicate of this bug. ***

Comment 13

[Pierre Saslawsky]

Reassigning peterl's bugs to myself.

Comment 14

[Pierre Saslawsky]

Accepting peterl's bugs that have a Target Milestone

Comment 15

[Pierre Saslawsky]

Pushing my M15 bugs to M16

Comment 16

[Jesse Ruderman]

Current behavior: all colors are inverted wherever text is selected, including 
background colors and even background images (including animated ones).  This 
is bad because selecting is supposed to give the user a way to force the text 
to be readable, and inverting colors often doesn't do much to make text with 
bad contrast readable.

A semi-random page that shows this effect with a background image is 
http://www.iwaynet.net/~karinaoi/karin.html .  A file 
called /BIN/res/samples/test2.html (debug->verification->jpeg images) included 
with the current build shows this with an animated background image.

Comment 17

[Pierre Saslawsky]

How come this bug is still on my list? Reassigned to mjudge, module owner for 
Selection. The selection should use the CSS System Colors but beware, they may 
not be all implemented in the platform-specific GFX libs.

Comment 18

[Kathleen :Brade]

I think Mike is planning toork on this for M15 if he has a chance.

Comment 19

[mjudge]

I am hooking up look and feel right now but pierre will do the style work to 
allow me to ask for the current colors to draw. look and feel should be done 
today.

Comment 20

[mjudge]

this is fixed. style also needs to be added but that is not the scope of this 
bug i dont think ;) no more green selection ya! ;)

Comment 21

[Blake Ross]

*SPAM*: Changing the QA contact of all open/resolved Selection bugs from 
[e-mail] to [e-mail].  After the many great years of service 
Eli has given to Mozilla, it's time for him to move on; he has accepted a 
position at Eazel.  We'll be sad to see him go, and I'll do my best to fill his 
spot...

Comment 22

[timeless]

vrfy w2k 9/25-5 talkback for the basic test page in 2000-03-11 21:42, and gneral
browsing.
a solaris intel pr2 build behaves the same.

could someone w/ a mac verify this?

Comment 23

[Asa Dotzler [:asa]]

verified Fixed on Mac 092808 mozilla trunk build.

Comment 24

[Diego Biurrun]

Verified on WinME with milestone 0.9.3.

Somebody on Linux needs to verify this or can it be marked verified anyway?

Comment 25

[Diego Biurrun]

VERIFIED on Linux 0.9.5 and Gnome 1.2. We need a restart to catch theme changes,
but after that it works.