sweep in behind popups

Author: junkdog

src/client/config.rs

@@ -121,7 +121,7 @@ pub enum PipelineStatus {
 impl Default for PollingConfig {
     fn default() -> Self {
         Self {
-            projects_interval: Duration::from_secs(60),
+            projects_interval: Duration::from_secs(15),
             jobs_interval: Duration::from_secs(30),
         }
     }

src/effect_registry.rs

@@ -184,11 +184,17 @@ impl EffectRegistry {
     /// # Returns
     ///
     /// A parallel effect combining coalescing and left-to-right sweep animation
-    pub fn register_projects_table_new_data(&mut self) {
+    pub fn register_projects_table_new_data(&mut self, exclude_popup_area: Option<RefRect>) {
+        let filter = match exclude_popup_area {
+            Some(area) => AllOf(vec![Inner(Margin::new(1, 1)), Not(ref_rect_filter(area).into())]),
+            None => Inner(Margin::new(1, 1)),
+        };
+
         let fx = parallel(&[
             coalesce(550),
             sweep_in(Motion::LeftToRight, 50, 0, Dark0Hard, (450, Interpolation::QuadIn)),
-        ]);
+        ])
+        .with_filter(filter);
 
         self.effects.add_effect(fx);
     }
@@ -493,7 +499,7 @@ fn dim_screen_behind_popup(screen_area: RefRect, popup_area: RefRect) -> Effect
 /// # Returns
 ///
 /// A `CellFilter` that matches cells within the specified rectangle
-fn ref_rect_filter(ref_rect: RefRect) -> CellFilter {
+pub fn ref_rect_filter(ref_rect: RefRect) -> CellFilter {
     CellFilter::PositionFn(ref_count(Box::new(move |pos| ref_rect.contains(pos))))
 }
 

src/ui/popup/config_popup.rs

@@ -29,7 +29,7 @@ pub struct ConfigPopupState {
     pub cursor_position: Position,
     input_fields: Vec<InputField>,
     pub error_message: Option<CompactString>,
-    popup_area: RefRect,
+    pub popup_area: RefRect,
 }
 
 impl ConfigPopup {

src/ui/popup/pipeline_actions_popup.rs

@@ -18,7 +18,7 @@ pub struct PipelineActionsPopup {}
 pub struct PipelineActionsPopupState {
     pub actions: Vec<GlimEvent>,
     pub list_state: ListState,
-    popup_area: RefRect,
+    pub popup_area: RefRect,
 }
 
 impl PipelineActionsPopupState {

src/ui/popup/project_details_popup.rs

@@ -27,7 +27,7 @@ pub struct ProjectDetailsPopupState {
     project_stat_summary: Text<'static>,
     pub pipelines: PipelineTable, // widget
     pub pipelines_table_state: TableState,
-    popup_area: RefRect,
+    pub popup_area: RefRect,
 }
 
 impl ProjectDetailsPopup {

src/ui/stateful_widgets.rs

@@ -54,7 +54,10 @@ impl StatefulWidgets {
         match event {
             GlimEvent::ProjectNext => self.handle_project_selection(1, app),
             GlimEvent::ProjectPrevious => self.handle_project_selection(-1, app),
-            GlimEvent::ProjectsLoaded(_) => effects.register_projects_table_new_data(),
+            GlimEvent::ProjectsLoaded(_) => {
+                let exclude_area_from_fx = self.current_popup_area();
+                effects.register_projects_table_new_data(exclude_area_from_fx)
+            },
             GlimEvent::ProjectDetailsOpen(id) => {
                 let popup_area = RefRect::default();
                 effects.register_project_details(popup_area.clone());
@@ -89,6 +92,14 @@ impl StatefulWidgets {
         }
     }
 
+    fn current_popup_area(&self) -> Option<RefRect> {
+        if let Some(pd) = &self.project_details {
+            Some(pd.popup_area.clone())
+        } else if let Some(cp) = &self.config_popup_state {
+            Some(cp.popup_area.clone())
+        } else { self.pipeline_actions.as_ref().map(|pa| pa.popup_area.clone()) }
+    }
+
     fn refresh_project_details(&mut self, project: &Project) {
         let requires_refresh = self
             .project_details