Skip to content

Commit

Permalink
style: Animate the 'normal' value of font-style as 'oblique 0deg'
Browse files Browse the repository at this point in the history
This matches what the spec[1] says for font-style:

> Animation type: by computed value type; normal animates as oblique 0deg

A bunch of WPT tests for font-style animation are landing in web-platform-tests/wpt#37570.
Current Gecko passes 66/129 of the testcases there; with this patch applied it passes all the tests.

[1] https://drafts.csswg.org/css-fonts-4/#font-style-prop

Differential Revision: https://phabricator.services.mozilla.com/D166128
  • Loading branch information
jfkthame authored and Loirooriol committed Nov 3, 2023
1 parent fadaaa1 commit 0867aa8
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions components/style/values/computed/font.rs
Expand Up @@ -1005,7 +1005,10 @@ impl ToAnimatedValue for FontStyle {
#[inline]
fn to_animated_value(self) -> Self::AnimatedValue {
if self == Self::NORMAL {
return generics::FontStyle::Normal;
// This allows us to animate between normal and oblique values. Per spec,
// https://drafts.csswg.org/css-fonts-4/#font-style-prop:
// Animation type: by computed value type; 'normal' animates as 'oblique 0deg'
return generics::FontStyle::Oblique(Angle::from_degrees(0.0))
}
if self == Self::ITALIC {
return generics::FontStyle::Italic;
Expand All @@ -1018,7 +1021,13 @@ impl ToAnimatedValue for FontStyle {
match animated {
generics::FontStyle::Normal => Self::NORMAL,
generics::FontStyle::Italic => Self::ITALIC,
generics::FontStyle::Oblique(ref angle) => Self::oblique(angle.degrees()),
generics::FontStyle::Oblique(ref angle) =>
if angle.degrees() == 0.0 {
// Reverse the conversion done in to_animated_value()
Self::NORMAL
} else {
Self::oblique(angle.degrees())
},
}
}
}
Expand Down

0 comments on commit 0867aa8

Please sign in to comment.