diff options
author | Boris Chiou <boris.chiou@gmail.com> | 2017-08-10 17:41:47 +0800 |
---|---|---|
committer | Boris Chiou <boris.chiou@gmail.com> | 2017-08-14 11:21:31 +0800 |
commit | 02277ce4a191c56359ebe2c033f59dec4423ad42 (patch) | |
tree | 746f2dfbe1b95c8a6440d3077d36e872b5e645ae | |
parent | 0eabba3c6fa232fd732eb2127f8fe285efd04288 (diff) | |
download | servo-02277ce4a191c56359ebe2c033f59dec4423ad42.tar.gz servo-02277ce4a191c56359ebe2c033f59dec4423ad42.zip |
Fix the conversion from Perspective into ComputedMatrix.
We convert a perspective function into a ComputedMatrix, but we apply the
perspective length to a wrong matrix item. We should put it into m34,
instead of m43. m43 is for translate parameter Z.
-rw-r--r-- | components/style/properties/helpers/animated_properties.mako.rs | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/components/style/properties/helpers/animated_properties.mako.rs b/components/style/properties/helpers/animated_properties.mako.rs index 4b423751d6b..2f11d506f29 100644 --- a/components/style/properties/helpers/animated_properties.mako.rs +++ b/components/style/properties/helpers/animated_properties.mako.rs @@ -1474,11 +1474,17 @@ fn add_weighted_transform_lists(from_list: &[TransformOperation], } } (&TransformOperation::Perspective(fd), - &TransformOperation::Perspective(_td)) => { + &TransformOperation::Perspective(td)) => { let mut fd_matrix = ComputedMatrix::identity(); let mut td_matrix = ComputedMatrix::identity(); - fd_matrix.m43 = -1. / fd.to_f32_px(); - td_matrix.m43 = -1. / _td.to_f32_px(); + if fd.0 > 0 { + fd_matrix.m34 = -1. / fd.to_f32_px(); + } + + if td.0 > 0 { + td_matrix.m34 = -1. / td.to_f32_px(); + } + let sum = fd_matrix.add_weighted(&td_matrix, self_portion, other_portion) .unwrap(); result.push(TransformOperation::Matrix(sum)); |