482692 - Should be able to override -moz-border-colors with border-color

Status: RESOLVED FIXED

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

Description

[Dão Gottwald [:dao]]

Take http://mxr.mozilla.org/mozilla-central/source/toolkit/themes/pinstripe/global/textbox.css for example. It uses -moz-border-*-colors, and whenever you use -moz-appearance:none to manually style a particular textbox, you have to use -moz-border-*-colors even if you only want a single color on each side. That's annoying.

Comment 1

[David Baron :dbaron:]

Making border-left-color, etc., into shorthands would be a good bit of a pain.  I'm not sure if it's worth it.

I'd note that you can use -moz-border-*-colors: none to turn off that behavior and use border-*-color again.

Comment 2

[Dão Gottwald [:dao]]

(In reply to comment #1)
> I'd note that you can use -moz-border-*-colors: none to turn off that behavior
> and use border-*-color again.

But I can't do:

  -moz-border-colors: none;
  border: 1px solid red;

since there's no -moz-border-colors shorthand, right?
So this is hardly better:

  -moz-border-top-colors: none;
  -moz-border-right-colors: none;
  -moz-border-bottom-colors: none;
  -moz-border-left-colors: none;
  border: 1px solid red;

than:

  border: 1px solid;
  -moz-border-top-colors: red;
  -moz-border-right-colors: red;
  -moz-border-bottom-colors: red;
  -moz-border-left-colors: red;


Even if there was a -moz-border-colors shorthand, I'm not sure that "none" is helpful, because this:

  -moz-border-colors: none;
  border: 1px solid red;

doesn't seem like a great improvement over this:

  border: 1px solid;
  -moz-border-colors: red;

when I really want this to just work:

  border: 1px solid red;

Comment 3

[David Baron :dbaron:]

So one thing that would be quite easy is to make the 'border' shorthand imply -moz-border-*-colors: none, and likewise make border-X imply -moz-border-X-colors: none.  It seems like that would help your problem a good bit; not sure how much it would break.

Comment 4

[Dão Gottwald [:dao]]

Yeah, that would help, but...

> make border-X imply -moz-border-X-colors: none

what about -moz-border-start/end?

Comment 5

[David Baron :dbaron:]

To do that I'd have to implement -moz-border-start-colors and -moz-border-end-colors, I think.

Comment 6

[David Baron :dbaron:]

(In reply to comment #3)
> and likewise make border-X imply -moz-border-X-colors: none

Actually, this part is harder than I thought, thanks to the fun of the parser and the differences between left/right and top/bottom (thanks to start/end).

Comment 7

[David Baron :dbaron:]

Attachment: patch

Here's a patch for making just the 'border' shorthand reset -moz-border-*-colors and (per last week's CSS WG decision) 'border-image'.

Comment 8

[Dão Gottwald [:dao]]

(In reply to comment #7)
> Created an attachment (id=366883) [details]
> patch
> 
> Here's a patch for making just the 'border' shorthand reset
> -moz-border-*-colors and (per last week's CSS WG decision) 'border-image'.

Can we get this reviewed and landed?

Comment 9

[Boris Zbarsky [:bzbarsky]]

I'd sort of like to know whether dbaron thinks the patch is ready to go before reviewing.

Comment 10

[David Baron :dbaron:]

Yeah, the patch is fine, but it could use a test.

Comment 11

[David Baron :dbaron:]

Attachment: patch

with test

Comment 12

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 427596 [details] [diff] [review]
patch

r=bzbarsky

Comment 13

[David Baron :dbaron:]

I had to make a few additional test changes:
http://hg.mozilla.org/users/dbaron_mozilla.com/patches/rev/2b70680896ee

Comment 14

[David Baron :dbaron:]

http://hg.mozilla.org/mozilla-central/rev/0e7b78eabfd8

Comment 15

[[e-mail]]

Does or should this affect serialization at all?

Comment 16

[mcdavis941 (sporadically reading bugmail)]

(In reply to comment #7)
> and (per last week's CSS WG decision) 'border-image'.

Is there a link to that particular CSS WG discussion/decision available, for reference?