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

RESOLVED FIXED in Firefox 5

Status

()

defect
P3
normal
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

Trunk
mozilla5
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox5 fixed)

Details

Attachments

(1 attachment)

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
Posted 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: 8 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.