Convert scrollbar_width from u8 to f32

Author: nicoburns

scripts/gentest/src/main.rs

@@ -391,10 +391,7 @@ fn generate_node(ident: &str, node: &Value) -> TokenStream {
         let overflow_x = overflow_x.unwrap_or(quote!(taffy::style::Overflow::Visible));
         let overflow_y = overflow_y.unwrap_or(quote!(taffy::style::Overflow::Visible));
         let overflow = quote!(overflow: taffy::geometry::Point { x: #overflow_x, y: #overflow_y },);
-        let scrollbar_width = quote_number_prop("scrollbar_width", style, |value: f32| {
-            let value = value as u8;
-            quote!(#value)
-        });
+        let scrollbar_width = quote_number_prop("scrollbar_width", style, |value: f32| quote!(#value));
         (overflow, scrollbar_width)
     } else {
         (quote!(), quote!())

src/compute/flexbox.rs

@@ -435,7 +435,7 @@ fn compute_constants(
     // However, the axis are switched (transposed) because a node that scrolls vertically needs
     // *horizontal* space to be reserved for a scrollbar
     let scrollbar_gutter = style.overflow.transpose().map(|overflow| match overflow {
-        Overflow::Scroll => style.scrollbar_width as f32,
+        Overflow::Scroll => style.scrollbar_width,
         _ => 0.0,
     });
     // TODO: make side configurable based on the `direction` property

src/style/mod.rs

@@ -164,7 +164,7 @@ pub struct Style {
     /// How children overflowing their container should affect layout
     pub overflow: Point<Overflow>,
     /// How much space (in points) should be reserved for the scrollbars of `Overflow::Scroll` and `Overflow::Auto` nodes.
-    pub scrollbar_width: u8,
+    pub scrollbar_width: f32,
 
     // Position properties
     /// What should the `position` value of this struct use as a base offset?
@@ -277,7 +277,7 @@ impl Style {
     pub const DEFAULT: Style = Style {
         display: Display::DEFAULT,
         overflow: Point { x: Overflow::Visible, y: Overflow::Visible },
-        scrollbar_width: 0,
+        scrollbar_width: 0.0,
         position: Position::Relative,
         inset: Rect::auto(),
         margin: Rect::zero(),
@@ -350,7 +350,7 @@ mod tests {
         let old_defaults = Style {
             display: Default::default(),
             overflow: Default::default(),
-            scrollbar_width: 0,
+            scrollbar_width: 0.0,
             position: Default::default(),
             #[cfg(feature = "flexbox")]
             flex_direction: Default::default(),
@@ -467,6 +467,6 @@ mod tests {
         assert_type_size::<Line<GridPlacement>>(8);
 
         // Overall
-        assert_type_size::<Style>(344);
+        assert_type_size::<Style>(352);
     }
 }