Invert ignore_focus to allow_focus

crates/egui/src/containers/area.rs

@@ -471,7 +471,7 @@ impl Area {
                     sense,
                     enabled,
                 },
-                false,
+                true,
             );
 
             if movable && move_response.dragged() {

crates/egui/src/containers/window.rs

@@ -842,7 +842,7 @@ fn resize_interaction(
                 sense: Sense::drag(),
                 enabled: true,
             },
-            false,
+            true,
         );
         SideResponse {
             hover: response.hovered(),

crates/egui/src/context.rs

@@ -1049,8 +1049,11 @@ impl Context {
     /// You should use [`Ui::interact`] instead.
     ///
     /// If the widget already exists, its state (sense, Rect, etc) will be updated.
+    ///
+    /// `allow_focus` should usually be true, unless you call this function multiple times with the
+    /// same widget, then `allow_focus` should only be true once (like in [`Ui::new`] (true) and [`Ui::interact_bg`] (false)).
     #[allow(clippy::too_many_arguments)]
-    pub(crate) fn create_widget(&self, w: WidgetRect, ignore_focus: bool) -> Response {
+    pub(crate) fn create_widget(&self, w: WidgetRect, allow_focus: bool) -> Response {
         // Remember this widget
         self.write(|ctx| {
             let viewport = ctx.viewport();
@@ -1060,12 +1063,12 @@ impl Context {
             // but also to know when we have reached the widget we are checking for cover.
             viewport.this_frame.widgets.insert(w.layer_id, w);
 
-            if w.sense.focusable && !ignore_focus {
+            if allow_focus && w.sense.focusable {
                 ctx.memory.interested_in_focus(w.id);
             }
         });
 
-        if !w.enabled || !w.sense.focusable || !w.layer_id.allow_interaction() && !ignore_focus {
+        if allow_focus && (!w.enabled || !w.sense.focusable || !w.layer_id.allow_interaction()) {
             // Not interested or allowed input:
             self.memory_mut(|mem| mem.surrender_focus(w.id));
         }
@@ -1078,7 +1081,7 @@ impl Context {
         let res = self.get_response(w);
 
         #[cfg(feature = "accesskit")]
-        if w.sense.focusable && !ignore_focus {
+        if allow_focus && w.sense.focusable {
             // Make sure anything that can receive focus has an AccessKit node.
             // TODO(mwcampbell): For nodes that are filled from widget info,
             // some information is written to the node twice.

crates/egui/src/response.rs

@@ -866,7 +866,7 @@ impl Response {
                 sense: self.sense | sense,
                 enabled: self.enabled,
             },
-            false,
+            true,
         )
     }
 

crates/egui/src/ui.rs

@@ -162,7 +162,7 @@ impl Ui {
                 sense,
                 enabled: ui.enabled,
             },
-            false,
+            true,
         );
 
         if disabled {
@@ -295,7 +295,7 @@ impl Ui {
                 sense,
                 enabled: child_ui.enabled,
             },
-            false,
+            true,
         );
 
         child_ui
@@ -973,7 +973,7 @@ impl Ui {
                 sense,
                 enabled: self.enabled,
             },
-            false,
+            true,
         )
     }
 
@@ -1011,7 +1011,7 @@ impl Ui {
                 sense: self.sense,
                 enabled: self.enabled,
             },
-            true,
+            false,
         )
     }