Closed Bug 654890 Opened 10 years ago Closed 10 years ago

[css3-animations] -moz-animation-play-state should not be part of -moz-animation shorthand

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla5
Tracking Status
firefox5 --- fixed

People

(Reporter: dbaron, Assigned: dbaron)

References

Details

Attachments

(1 file)

I didn't notice that -moz-animation-play-state wasn't part of the -moz-animation shorthand, so I implemented it as part of the shorthand.

It's currently being discussed on www-style, so it might change, though:
http://lists.w3.org/Archives/Public/www-style/2011May/0082.html
Attached patch patchSplinter Review
Attachment #530792 - Flags: review?(bzbarsky)
Comment on attachment 530792 [details] [diff] [review]
patch

r=me
Attachment #530792 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/mozilla-central/rev/2054cc185d8b
Status: NEW → RESOLVED
Closed: 10 years ago
Priority: -- → P3
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
Comment on attachment 530792 [details] [diff] [review]
patch

This is a very straightforward patch to our animations support to match the spec and WebKit; just something that I misread in the spec.  It has good test coverage, so I think it's very low risk.
Attachment #530792 - Flags: approval-mozilla-aurora?
Attachment #530792 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Please land for Aurora by Monday May 16 or the approval will potentially be lost.  Please mark as status-firefox5 fixed when you do.
Target Milestone: mozilla6 → mozilla5
Assignee: nobody → dbaron
Blocks: 1037314
You need to log in before you can comment on or make changes to this bug.