Summary: | CSS transitions should work on the flex property | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tony Chang <tony> | ||||||
Component: | Layout and Rendering | Assignee: | Tony Chang <tony> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | dino, hyatt, morrita, ojan, simon.fraser, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 62048 | ||||||||
Attachments: |
|
Description
Tony Chang
2012-01-09 17:09:53 PST
Created attachment 125449 [details]
Patch
It looks the test compares the value as a number (float), not a tuple of the numbers. Is it intentional? (Disclaimer: I'm not a flexbox expert.) Comment on attachment 125449 [details]
Patch
The spec is changing so this patch isn't correct anymore. I'll post a new patch when we're ready.
Created attachment 135668 [details]
Patch
Patch looks good to me. Someone with transitions experience should probably r+ though. Comment on attachment 135668 [details]
Patch
Cool!
Comment on attachment 135668 [details] Patch Clearing flags on attachment: 135668 Committed r113264: <http://trac.webkit.org/changeset/113264> All reviewed patches have been landed. Closing bug. |