Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-05 13:40:08 +01:00
Code Issues Projects Releases Packages Wiki Activity

style: Don't make visibility additive.

Browse source 
As per discussion here: https://github.com/web-platform-tests/wpt/pull/19160

This property type does not have a procedure for addition defined so it should
not be additive.

Differential Revision: https://phabricator.services.mozilla.com/D48454
This commit is contained in:
Brian Birtles 2019-10-08 01:27:29 +00:00 • committed by Emilio Cobos Álvarez
parent 38e5897353
commit f6b587051d
1 changed files with 13 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

21
components/style/properties/helpers/animated_properties.mako.rs
View file

@ -768,17 +768,22 @@ animated_list_impl!(<T> for crate::OwnedSlice<T>);
animated_list_impl!(<T> for SmallVec<[T; 1]>); animated_list_impl!(<T> for SmallVec<[T; 1]>);
animated_list_impl!(<T> for Vec<T>); animated_list_impl!(<T> for Vec<T>);
/// <https://drafts.csswg.org/css-transitions/#animtype-visibility> /// <https://drafts.csswg.org/web-animations-1/#animating-visibility>
impl Animate for Visibility { impl Animate for Visibility {
#[inline] #[inline]
fn animate(&self, other: &Self, procedure: Procedure) -> Result<Self, ()> { fn animate(&self, other: &Self, procedure: Procedure) -> Result<Self, ()> {
let (this_weight, other_weight) = procedure.weights(); match procedure {
match (*self, *other) { Procedure::Interpolate { .. } => {
(Visibility::Visible, _) => { let (this_weight, other_weight) = procedure.weights();
Ok(if this_weight > 0.0 { *self } else { *other }) match (*self, *other) {
}, (Visibility::Visible, _) => {
(_, Visibility::Visible) => { Ok(if this_weight > 0.0 { *self } else { *other })
Ok(if other_weight > 0.0 { *other } else { *self }) },
(_, Visibility::Visible) => {
Ok(if other_weight > 0.0 { *other } else { *self })
},
_ => Err(()),
}
}, },
_ => Err(()), _ => Err(()),
} }