The default bug view has changed. See this FAQ.

remove fallback CSS background color support

RESOLVED FIXED in mozilla1.9.2a1

Status

()

Core
CSS Parsing and Computation
P1
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

Trunk
mozilla1.9.2a1
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
The CSS WG decided to remove the fallback background color feature from css3-backgrounds (and potentially add an equivalent later using a new value type, e.g., an image() function that takes fallback).

We should remove the implementation that was added in bug 322475.
(Assignee)

Updated

8 years ago
Blocks: 482708
(Assignee)

Comment 1

8 years ago
Created attachment 381965 [details] [diff] [review]
patch
Attachment #381965 - Flags: superreview?(bzbarsky)
Attachment #381965 - Flags: review?(bzbarsky)
:(

This seems like a pretty good feature anyhow, perhaps just change it to a -moz prefixed solution for now?

I think some ui features rely on this, cc'ing dao to weigh in here.
(Assignee)

Comment 3

8 years ago
This feature is not in 3.5; I don't think we've done much UI development that's for post-3.5-only.  Furthermore, there are much nicer ways to have syntax for the same feature.
Yeah, we haven't used this yet.
Comment on attachment 381965 [details] [diff] [review]
patch

r+sr=bzbarsky
Attachment #381965 - Flags: superreview?(bzbarsky)
Attachment #381965 - Flags: superreview+
Attachment #381965 - Flags: review?(bzbarsky)
Attachment #381965 - Flags: review+
(Assignee)

Comment 6

8 years ago
http://hg.mozilla.org/mozilla-central/rev/37f8a33e4c54
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Priority: -- → P1
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1

Comment 7

8 years ago
This fixed the red chin in acid2 right?

Detailed at http://whereswalden.com/2009/02/20/when-good-tests-go-bad-firefox-on-acid2/
(Assignee)

Comment 8

8 years ago
Yes.
Keywords: dev-doc-needed
(Assignee)

Comment 9

8 years ago
I don't think any documentation is needed here, since we removed a feature that we never shipped in the first place (so that we wouldn't ship it).
The implementation in bug 322475 was documented, so I need to make sure the background color fallback support isn't part of what got documented. Hence the addition of the doc-needed note here.
OK, after some reviewing of the docs, this doesn't in fact require any tweaks to the docs, so removing the tag.
Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.