Closed Bug 188972 Opened 22 years ago Closed 8 years ago

-moz-use-text-color is sometimes serialized for the border shorthand

Categories

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

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 376075

People

(Reporter: vladimire, Assigned: glazou)

Details

Attachments

(2 files, 1 obsolete file)

If I set border property on one div to dotted, and set border-top property on
another div to dotted. When I get the values of borderTop of both divs from
javascript I get:
test0.style.borderTop = medium dotted -moz-use-text-color
test1.style.borderTop = medium dotted 

Which is inconsistent. There is no color for the second borderTop. I dont know
what is correct, to show the color, or not to show the color, but the two cases
should be consistent. Probably the right behaviour would be what IE does, just
show "dotted" because other properties are default.
Attached file testcase
glazou, the shorthand stuff needs to drop that color value...
Reassigning to glazman, Daniel, hand this back if you don't want it.
Assignee: jst → glazman
Attached patch fix #1 (obsolete) — Splinter Review
Comment on attachment 112753 [details] [diff] [review]
rogggnnnnntdddjjjuuuuu, fix #2

kin, can you r= please ?
jst, can you sr= please ?
(given that bz seems to drop reviews...)
Attachment #112753 - Flags: superreview?(jst)
Attachment #112753 - Flags: review?(kin)
Attachment #112753 - Flags: superreview?(jst)
Attachment #112753 - Flags: superreview+
Attachment #112753 - Flags: review?(kin)
Attachment #112753 - Flags: review+
Daniel, plans to land this patch? It looks like it may have bitrotted a bit,
though....
This looks as if it would make the style string end in a space :-(
Hmm... that it could do, yes.  We probably want to avoid that...
QA Contact: ian → general
QA Contact: general → style-system
Summary: Inconsistent value for a property → -moz-use-text-color is sometimes serialized for the border shorthand
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: