Address code review

Author: PoignardAzur

masonry_core/src/core/contexts.rs

@@ -955,12 +955,14 @@ impl LayoutCtx<'_> {
     /// [`post_paint`]: Widget::post_paint
     /// [clip shape]: crate::doc::masonry_concepts#clip-shape.
     pub fn set_clips_contents(&mut self, clips: bool) {
-        self.widget_state.clips_contents = clips;
+        if self.widget_state.clips_contents != clips {
+            self.widget_state.clips_contents = clips;
 
-        self.widget_state.request_accessibility = true;
-        self.widget_state.needs_accessibility = true;
-        self.widget_state.needs_paint = true;
-        self.global_state.needs_pointer_pass = true;
+            self.widget_state.request_accessibility = true;
+            self.widget_state.needs_accessibility = true;
+            self.widget_state.needs_paint = true;
+            self.global_state.needs_pointer_pass = true;
+        }
     }
 
     // TODO - Add set_clip_shape(impl Shape) method
@@ -1124,7 +1126,7 @@ impl_context_method!(
             border_box_baseline - self.widget_state.border_box_insets.y1
         }
 
-        /// Whether the children clips its own contents and that of its children.
+        /// Whether the widget clips its own contents and that of its children.
         ///
         /// For more information, see
         /// [`LayoutCtx::set_clips_contents`](crate::core::LayoutCtx::set_clips_contents).

masonry_core/src/doc/masonry_concepts.md

@@ -221,7 +221,7 @@ Widgets have a shape, usually one that matches their visual appearance, which ha
 - If the widget is set to clip its contents, pointer events outside the clip shape won't affect the children either.
 - If the widget is set to clip its contents, its scene and the children's scenes will be painted inside of the clip shape.
 
-Currently, the clip shape is hardcoded to be the layout rect.
+Currently, the clip shape is hardcoded to be a rect with the widget's size and position.
 
 <!-- TODO: Rename to "widget shape" instead? -->
 <!-- Need a better name. -->