fmt and clippy

Fix collapse action

crates/ecolor/src/rgba.rs

@@ -8,7 +8,7 @@ use crate::{
 #[derive(Clone, Copy, Debug, Default, PartialEq)]
 #[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
 #[cfg_attr(feature = "bytemuck", derive(bytemuck::Pod, bytemuck::Zeroable))]
-pub struct Rgba(pub [f32; 4]);
+pub struct Rgba(pub(crate) [f32; 4]);
 
 impl std::ops::Index<usize> for Rgba {
     type Output = f32;

crates/egui/src/containers/collapsing_header.rs

@@ -66,14 +66,8 @@ impl CollapsingState {
         match action {
             WindowAction::Show => self.state.shown = true,
             WindowAction::Hide => self.state.shown = false,
-            WindowAction::Expand => {
-                self.state.shown = true;
-                self.state.expanded = true;
-            }
-            WindowAction::Collapse => {
-                self.state.shown = true;
-                self.state.expanded = true;
-            }
+            WindowAction::Expand => self.state.expanded = true,
+            WindowAction::Collapse => self.state.expanded = false,
             WindowAction::ToggleShow => {
                 self.state.shown = !self.state.shown;
             }
@@ -309,7 +303,7 @@ pub struct HeaderResponse<'ui, HeaderRet> {
 }
 
 impl<'ui, HeaderRet> HeaderResponse<'ui, HeaderRet> {
-    #[deprecated = "Use het `HeaderResponse::is_expanded`"]
+    #[deprecated = "Use the `HeaderResponse::is_expanded` instead"]
     pub fn is_open(&self) -> bool {
         self.is_expanded()
     }

crates/egui/src/containers/window.rs

@@ -91,8 +91,8 @@ impl<'open> Window<'open> {
         self
     }
 
-    /// Call this to programmatically expand, collapse, show, hide the window.
-    /// The Option value will be taken, so the option will be set to None.
+    /// Call this to programmatically expand/collapse or to show/hide the window.
+    /// The passed Option value will be taken, so the option will be set to None.
     #[inline]
     pub fn window_action(mut self, action: &mut Option<WindowAction>) -> Self {
         self.window_action = action.take();
@@ -477,8 +477,6 @@ impl<'open> Window<'open> {
         // Add border padding to the inner margin to prevent it from covering the contents
         window_frame.inner_margin += border_padding;
 
-        //let is_explicitly_closed = matches!(open, Some(false));
-        //let is_open = !is_explicitly_closed || ctx.memory(|mem| mem.everything_is_visible());
         let all_visible = ctx.memory(|mem| mem.everything_is_visible());
         let area_id = area.id;
 

crates/egui_demo_lib/src/demo/about.rs

@@ -1,5 +1,5 @@
-use egui::WindowAction;
 use crate::demo::Demo;
+use egui::WindowAction;
 
 #[derive(Default)]
 #[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]
@@ -25,7 +25,12 @@ impl crate::Demo for About {
 }
 
 impl About {
-    pub fn show_action(&mut self, ctx: &egui::Context, open: &mut bool, action: &mut Option<WindowAction>) {
+    pub fn show_action(
+        &mut self,
+        ctx: &egui::Context,
+        open: &mut bool,
+        action: &mut Option<WindowAction>,
+    ) {
         egui::Window::new(self.name())
             .default_width(320.0)
             .default_height(480.0)
@@ -38,7 +43,6 @@ impl About {
                 self.ui(ui);
             });
     }
-
 }
 
 impl crate::View for About {

crates/egui_demo_lib/src/demo/demo_app_windows.rs

@@ -296,7 +296,8 @@ impl DemoWindows {
 
     /// Show the open windows.
     fn show_windows(&mut self, ctx: &Context) {
-        self.about.show_action(ctx, &mut self.about_is_open, &mut self.about_window_action);
+        self.about
+            .show_action(ctx, &mut self.about_is_open, &mut self.about_window_action);
         self.demos.windows(ctx);
         self.tests.windows(ctx);
     }