666243 - Definition of border overwrites -moz-border-*-colors

Status: RESOLVED FIXED

(Core :: DOM: CSS Object Model, defect)

Description

[Sebastian Zartner [:sebo]]

User-Agent:       Mozilla/5.0 (Windows NT 6.1; rv:5.0) Gecko/20100101 Firefox/5.0
Build Identifier: 

When there's a 'border' property defined for a rule, any -moz-border-*-colors properties are not shown inside the cssText property of a CSSStyleRule object.

Reproducible: Always

Steps to Reproduce:
See the attached test case.


Expected Results:  
Both, 'border' and '-moz-border-*-colors' should be inside the cssText.

Comment 1

[Sebastian Zartner [:sebo]]

Attachment: Test case for bug 666243

Comment 2

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

This is as-intended.  The 'border' shorthand overwrites the -moz-border-*-colors properties; see discussion in bug 482692.

Comment 3

[Sebastian Zartner [:sebo]]

I see. Though overwriting '-moz-border-*-colors' with the values of 'border' is causing some side effects.
Imagine you create a style 'border' for a node inside the Style side panel of Firebug. After that you create a '-moz-border-top-colors' style.
What happens: The '-moz-border-top-colors' style just disappears and the user wonders why. This happened to my workmate yesterday and he thought it would be a bug in Firebug.
Maybe the problem here is, that the different border colors are not appearing in the shorthand property 'border'. Note, that the test case described above is working, if you define 'border: 1px solid;' and afterwards 'border-color: blue;'. The result will be 'border: 1px solid blue;'. So that behavior of '-moz-border-*-colors' is inconsistent.

Comment 4

[Sebastian Zartner [:sebo]]

Attachment: Enhanced test case for bug 666243

This is definitely a bug.
I enhanced the test case description to show what I mean.

Comment 5

[Sebastian Zartner [:sebo]]

Attachment: Enhanced test case for bug 666243

The attachment didn't get added as text/html for some reason, so I added it again and manually specified test/html as MIME type.

Comment 6

[Sebastian Zartner [:sebo]]

Hmm, seems, there is also a bug in Bugzilla when attaching files. Even the manual setting of text/html didn't get recognized. Sorry for that. So just download the file and follow the steps. You will see the problem I described.

Comment 7

[Sebastian Zartner [:sebo]]

David, could you reproduce the problem?

Comment 8

[Wayne Mery (:wsmwk)]

(In reply to Sebastian Zartner from comment #7)
> David, could you reproduce the problem?

Comment 9

[Sebastian Zartner [:sebo]]

Attachment: Test case

I cannot reproduce this issue anymore. I added a new, simplier test case to confirm that.

Sebastian

Comment 10

[Sebastian Zartner [:sebo]]

This obviously got fixed at some point, so I close this issue.
The serialization doesn't work properly in the case a -moz-border-*-colors property is set, though, which should be addressed by bug 713613.

Sebastian

Comment 11

[Wayne Mery (:wsmwk)]

Thanks. We use WORKSFORME when we d ont know which specific patch/bug fixed the issue

Comment 12

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

It was fixed by https://hg.mozilla.org/mozilla-central/rev/55e6c2ca07e4

Comment 13

[Wayne Mery (:wsmwk)]

(In reply to David Baron [:dbaron] (UTC-7, busy Oct 7-9) (needinfo? for questions) from comment #12)
> It was fixed by https://hg.mozilla.org/mozilla-central/rev/55e6c2ca07e4

 Bug 713643