156002 - Border colors specified using CSS are not ignored when "use own colors" is set

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect, P2)

Description

[Laura Bottelier]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.0.0) Gecko/20020530
BuildID:    2002053012

Bad: The border color speicifed in CSS are not ignored when the accesibility
option to overide the page its colors is set.

Good: The borders hould be draw using the preferred foreground color specifed by
the user.

Reproducible: Always
Steps to Reproduce:
1. Turn on accesibility option 'use own colors'


Actual Results:  The border colors are still affected by CSS

Expected Results:  All borders have the same color; the one selected by the user

Comment 1

[Laura Bottelier]

Attachment: Testcase showing different border styles with different colors

Comment 2

[Bill Mason]

Verified in build 2002070310 PC/Win98.

Comment 3

[Aaron Leventhal]

-> style system

Comment 4

[Boris Zbarsky [:bzbarsky]]

Attachment: Patch to fix this

Comment 5

[Boris Zbarsky [:bzbarsky]]

I'm going to hold off on this till bryner lands his big change... 

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 90405 [details] [diff] [review]
Patch to fix this

since bryner is not planning to land anytime soon....

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 90405 [details] [diff] [review]
Patch to fix this

I'd prefer if you used -moz-use-color (I think that's what the default is
called) rather than strColor / 'inherit'.  This will work better when colors
are changed (as they should be for various UA-level things, such as borders on
images in links).

Comment 8

[Boris Zbarsky [:bzbarsky]]

Attachment: good catch

Comment 9

[Boris Zbarsky [:bzbarsky]]

fixed

Comment 10

[Madhur Bhatia]

verified