`939c9b8`

fix(ui): present surface to window after hover re-renders

Authored by mfwolffe <wolffemf@dukes.jmu.edu> 3 months ago

SHA: 939c9b84a9cf692e91797f137edf728f6e0e021a
Parents: 0e7907b
Tree: 9bdba65

2 changed files

Status	File	+	-
M	`garnotify/src/ui/popup.rs`	7	0
M	`garnotify/src/ui/popup_manager.rs`	5	5

          Ok(())
+     }
 +    /// Render and present to window (for updates like hover effects)
 +    pub fn render_and_present(&mut self) -> Result<()> {
 +        self.render()?;
 +        self.present()?;
 +        Ok(())
 +    }
++
      /// Get colors based on notification urgency
      fn get_urgency_colors(&self) -> (Color, Color, Color) {
          let urgency = &self.notification.hints.urgency;

  use std::collections::HashMap;
  use std::sync::Arc;
  use tokio::sync::mpsc;
 -use tracing::{debug, error, info, warn};
 +use tracing::{debug, info, warn};
  use x11rb::protocol::Event as X11Event;
  use crate::config::Config;
                  if let Some(&id) = self.window_to_notification.get(&e.event) {
                      if let Some(popup) = self.popups.get_mut(&id) {
                          popup.on_leave();
 -                        // Re-render to clear hover effects
 -                        if let Err(e) = popup.render() {
 +                        // Re-render and present to clear hover effects
 +                        if let Err(e) = popup.render_and_present() {
                              warn!("Failed to re-render on leave: {}", e);
+                         }
+                     }
              X11Event::MotionNotify(e) => {
                  if let Some(&id) = self.window_to_notification.get(&e.event) {
                      if let Some(popup) = self.popups.get_mut(&id) {
 -                        // Update hover state, re-render if changed
 +                        // Update hover state, re-render and present if changed
                          if popup.on_motion(e.event_x as i32, e.event_y as i32) {
 -                            if let Err(err) = popup.render() {
 +                            if let Err(err) = popup.render_and_present() {
                                  warn!("Failed to re-render on motion: {}", err);
+                             }
+                         }