Revert "Changed logic for submenus closing (#3055)"

This reverts commit 128afdd.

crates/egui/src/menu.rs

@@ -618,6 +618,8 @@ impl MenuState {
         } else if !open && button.hovered() {
             let pos = button.rect.right_top();
             self.open_submenu(sub_id, pos);
+        } else if open && !button.hovered() && !self.hovering_current_submenu(&pointer) {
+            self.close_submenu();
         }
     }
 
@@ -642,6 +644,16 @@ impl MenuState {
         false
     }
 
+    /// Check if pointer is hovering current submenu.
+    fn hovering_current_submenu(&self, pointer: &PointerState) -> bool {
+        if let Some(sub_menu) = self.current_submenu() {
+            if let Some(pos) = pointer.hover_pos() {
+                return sub_menu.read().area_contains(pos);
+            }
+        }
+        false
+    }
+
     /// Cascade close response to menu root.
     fn cascade_close_response(&mut self, response: MenuResponse) {
         if response.is_close() {
@@ -673,4 +685,8 @@ impl MenuState {
             self.sub_menu = Some((id, Arc::new(RwLock::new(Self::new(pos)))));
         }
     }
+
+    fn close_submenu(&mut self) {
+        self.sub_menu = None;
+    }
 }