clapper-gtk: extra-menu-button: Drop popover focus on unmap

Fixes #496
Rafostar · Jan 18, 2025 · fe30ba2 · fe30ba2
1 parent 4472d87
commit fe30ba2
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/lib/clapper-gtk/clapper-gtk-extra-menu-button.c b/src/lib/clapper-gtk/clapper-gtk-extra-menu-button.c
@@ -574,6 +574,8 @@ popover_map_cb (GtkWidget *widget, ClapperGtkExtraMenuButton *self)
 
   GST_TRACE_OBJECT (self, "Popover map");
 
+  gtk_widget_set_can_focus (widget, TRUE);
+
   if (G_UNLIKELY (self->player == NULL))
     return;
 
@@ -608,6 +610,11 @@ popover_unmap_cb (GtkWidget *widget, ClapperGtkExtraMenuButton *self)
 
   GST_TRACE_OBJECT (self, "Popover unmap");
 
+  /* Drop focus after popover is closed. Fixes issue
+   * with keyboard shortcuts not working when closed
+   * while within submenu */
+  gtk_widget_set_can_focus (widget, FALSE);
+
   if (G_UNLIKELY (self->player == NULL))
     return;