Make style changes which alter inherited style properties request lay…

…out for all children

src/context.rs

@@ -51,6 +51,8 @@ pub struct ViewState {
     pub(crate) node: Node,
     pub(crate) children_nodes: Vec<Node>,
     pub(crate) request_layout: bool,
+    /// Layout is requested on all direct and indirect children.
+    pub(crate) request_layout_recursive: bool,
     pub(crate) has_style_selectors: StyleSelectors,
     pub(crate) viewport: Option<Rect>,
     pub(crate) layout_rect: Rect,
@@ -78,6 +80,7 @@ impl ViewState {
             layout_rect: Rect::ZERO,
             layout_props: Default::default(),
             request_layout: true,
+            request_layout_recursive: false,
             has_style_selectors: StyleSelectors::default(),
             animation: None,
             base_style: None,
@@ -370,6 +373,13 @@ impl AppState {
         }
     }
 
+    /// Requests layout for a view and all direct and indirect children.
+    pub(crate) fn request_layout_recursive(&mut self, id: Id) {
+        let view = self.view_state(id);
+        view.request_layout_recursive = true;
+        self.request_layout(id);
+    }
+
     pub fn request_layout(&mut self, id: Id) {
         let view = self.view_state(id);
         if view.request_layout {

src/style.rs

@@ -556,10 +556,12 @@ impl Style {
         }
     }
 
-    pub(crate) fn apply_only_inherited(this: &mut Rc<Style>, over: &Style) {
-        let any_inherited = over.map.iter().any(|(p, _)| p.info.inherited);
+    pub(crate) fn any_inherited(&self) -> bool {
+        !self.classes.is_empty() || self.map.iter().any(|(p, _)| p.info.inherited)
+    }
 
-        if any_inherited || !over.classes.is_empty() {
+    pub(crate) fn apply_only_inherited(this: &mut Rc<Style>, over: &Style) {
+        if over.any_inherited() {
             let inherited = over
                 .map
                 .iter()

src/view.rs

@@ -197,16 +197,30 @@ pub trait View {
     fn layout_main(&mut self, cx: &mut LayoutCx) -> Node {
         cx.save();
 
+        let view_state = cx.app_state_mut().view_state(self.id());
+
         let view_style = self.view_style();
         let view_class = self.view_class();
-        let class = cx.app_state_mut().view_state(self.id()).class;
+        let class = view_state.class;
         let class_array;
         let classes = if let Some(class) = class {
             class_array = [class];
             &class_array[..]
         } else {
             &[]
         };
+
+        // Propagate layout requests to children if needed.
+        if view_state.request_layout_recursive {
+            view_state.request_layout_recursive = false;
+            view_state.request_layout = true;
+            for child in self.children() {
+                cx.app_state_mut()
+                    .view_state(child.id())
+                    .request_layout_recursive = true;
+            }
+        }
+
         cx.app_state.compute_style(
             self.id(),
             view_style,
@@ -253,9 +267,6 @@ pub trait View {
 
         cx.save();
 
-        cx.style.direct = cx.app_state_mut().get_computed_style(self.id()).clone();
-        Style::apply_only_inherited(&mut cx.style.current, &cx.style.direct);
-
         let layout = cx
             .app_state()
             .get_layout(self.id())

src/views/label.rs

@@ -196,15 +196,6 @@ impl View for Label {
             return None;
         }
 
-        if self.font.read(cx) | self.style.read(cx) {
-            self.text_layout = None;
-            self.available_text = None;
-            self.available_width = None;
-            self.available_text_layout = None;
-            self.set_text_layout();
-            cx.app_state_mut().request_layout(self.id());
-        }
-
         let layout = cx.get_layout(self.id()).unwrap();
         let style = cx.app_state_mut().get_builtin_style(self.id);
         let text_overflow = style.text_overflow();

src/views/scroll.rs

@@ -44,9 +44,9 @@ enum BarHeldState {
 style_class!(pub Handle);
 style_class!(pub Track);
 
-prop!(pub Rounded: bool { inherited } = cfg!(target_os = "macos"));
-prop!(pub Thickness: Px { inherited } = Px(10.0));
-prop!(pub Border: Px { inherited } = Px(0.0));
+prop!(pub Rounded: bool {} = cfg!(target_os = "macos"));
+prop!(pub Thickness: Px {} = Px(10.0));
+prop!(pub Border: Px {} = Px(0.0));
 
 prop_extracter! {
     ScrollStyle {

src/window_handle.rs

@@ -692,18 +692,28 @@ impl WindowHandle {
                     }
                     UpdateMessage::BaseStyle { id, style } => {
                         let state = cx.app_state.view_state(id);
+                        let old_any_inherited = state.style.any_inherited();
                         state.base_style = Some(style);
-                        cx.request_layout(id);
+                        if state.style.any_inherited() || old_any_inherited {
+                            cx.app_state.request_layout_recursive(id);
+                        } else {
+                            cx.request_layout(id);
+                        }
                     }
                     UpdateMessage::Style { id, style } => {
                         let state = cx.app_state.view_state(id);
+                        let old_any_inherited = state.style.any_inherited();
                         state.style = style;
-                        cx.request_layout(id);
+                        if state.style.any_inherited() || old_any_inherited {
+                            cx.app_state.request_layout_recursive(id);
+                        } else {
+                            cx.request_layout(id);
+                        }
                     }
                     UpdateMessage::Class { id, class } => {
                         let state = cx.app_state.view_state(id);
                         state.class = Some(class);
-                        cx.request_layout(id);
+                        cx.app_state.request_layout_recursive(id);
                     }
                     UpdateMessage::StyleSelector {
                         id,