15286 - selection needs to swap highlight foreground/background color based on background color

Status: RESOLVED DUPLICATE of bug 56314

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

Description

[buster]

Nav 4 has an algorithm that will swap foreground and background highlight color
if the background highlight and the document background color are within a small
tolerance of each other.  This allows the selection to be visible regardless of
the background.

Comment 1

[Eli Goldberg]

buster, could you possibly provide a URL in which this is an issue? (I thought
we're doing correctly what you're referring to --- e.g. how does this differ from
9960 or 13831?)

Thanks!

Comment 2

[buster]

this is probably a dup of 13831 or 9960

Comment 3

[Kathleen :Brade]

I'm guessing that [e-mail] intended this to be dup'ed of #9960 (which
has been marked fixed) which is for xor'ing the selection.  Please re-open if you
intended this bug to be something else.


*** This bug has been marked as a duplicate of 9960 ***

Comment 4

[Peter Linss]

No, this is meant for something else. The issue is when selection code uses
style data for selection colors it still needs to detect foreground/bachground
collisions and swap foreground/background (not xor).

Comment 5

[rubydoo123]

setting to M15, this can wait until after the beta push

Comment 6

[rubydoo123]

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

Comment 7

[mjudge]

until the spec in CSS3 allows us to use this system of letting the OS decide 
what the colors are depending on the current foreground and background I am 
leaving the deciding of color to CSS3.  This will probably mean that the DEFAULT 
style of CSS will be the default foreground andbackground of the OS. then any 
page that wishes to override this can do so.  Until CSS3 has some specs on how 
to in a XP way allow the system to be handed the current color and style then 
return the new background and foreground color, this will be the case. no XOR 
for now in other words.

Comment 8

[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 9

[Jesse Ruderman]

*** This bug has been marked as a duplicate of 56314 ***

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/453e1e6db45b
[fenix] For https://github.com/mozilla-mobile/fenix/issues/15286: Don't crash when separators are long-touched