Address code review

Author: PoignardAzur

masonry/src/doc/05_pass_system.md

@@ -106,12 +106,12 @@ It is called when new widgets are added to the tree, or existing widgets are rem
 
 It will call the `register_children()` widget method on container widgets whose children changed, then the `update()` method with the `WidgetAdded` event on new widgets.
 
-<!-- TODO - document UPDATE DISABLED --- -->
-<!-- TODO - document UPDATE STASHED --- -->
-<!-- TODO - document UPDATE FOCUS CHAIN --- -->
-<!-- TODO - document UPDATE FOCUS --- (document iteration order) -->
-<!-- TODO - document UPDATE SCROLL --- -->
-<!-- TODO - document UPDATE POINTER --- (document iteration order) -->
+<!-- TODO - document update disabled --- -->
+<!-- TODO - document update stashed --- -->
+<!-- TODO - document update focus chain --- -->
+<!-- TODO - document update focus --- (document iteration order) -->
+<!-- TODO - document update scroll --- -->
+<!-- TODO - document update pointer --- (document iteration order) -->
 
 ### Layout pass
 

masonry/src/event.rs

@@ -312,22 +312,6 @@ pub enum Update {
 }
 
 impl PointerEvent {
-    /// Create a [`PointerEvent::PointerLeave`] event with dummy values.
-    pub fn new_pointer_leave() -> Self {
-        // TODO - The fact we're creating so many dummy values might be
-        // a sign we should refactor that struct
-        let pointer_state = PointerState {
-            physical_position: Default::default(),
-            position: Default::default(),
-            buttons: Default::default(),
-            mods: Default::default(),
-            count: 0,
-            focus: false,
-            force: None,
-        };
-        Self::PointerLeave(pointer_state)
-    }
-
     /// Returns the [`PointerState`] of the event.
     pub fn pointer_state(&self) -> &PointerState {
         match self {
@@ -388,6 +372,25 @@ impl PointerEvent {
             Self::Pinch(_, _) => true,
         }
     }
+
+    /// Create a [`PointerEvent::PointerLeave`] event with dummy values.
+    ///
+    /// This is used internally to create synthetic `PointerLeave` events when pointer
+    /// capture is lost.
+    pub fn new_pointer_leave() -> Self {
+        // TODO - The fact we're creating so many dummy values might be
+        // a sign we should refactor that struct
+        let pointer_state = PointerState {
+            physical_position: Default::default(),
+            position: Default::default(),
+            buttons: Default::default(),
+            mods: Default::default(),
+            count: 0,
+            focus: false,
+            force: None,
+        };
+        Self::PointerLeave(pointer_state)
+    }
 }
 
 impl TextEvent {