chiark / gitweb /
Make the Disobedience login window a bit saner. There is now just a
[disorder] / disobedience / control.c
index 39bced1b46b82b9ccba33b7ed7c8d8a5eca02544..e1f44f28fcea9102aee9d42b1940575d5b154df5 100644 (file)
@@ -22,6 +22,7 @@
  */
 
 #include "disobedience.h"
+#include "mixer.h"
 
 /* Forward declarations ---------------------------------------------------- */
 
@@ -45,6 +46,8 @@ static void update_disable(const struct icon *);
 static void update_rtp(const struct icon *);
 static void update_nortp(const struct icon *);
 static void clicked_icon(GtkButton *, gpointer);
+static void clicked_menu(GtkMenuItem *, gpointer userdata);
+static void toggled_menu(GtkCheckMenuItem *, gpointer userdata);
 
 static int enable_rtp(disorder_eclient *c,
                       disorder_eclient_no_response *completed,
@@ -64,8 +67,8 @@ static gchar *format_balance(GtkScale *scale, gdouble value);
 
 /* Control bar ------------------------------------------------------------- */
 
-/** @brief Guard against feedback loop in volume control */
-static int suppress_set_volume;
+/** @brief Guard against feedback */
+int suppress_actions = 1;
 
 /** @brief Definition of an icon
  *
@@ -79,8 +82,8 @@ struct icon {
   /** @brief Text for tooltip */
   const char *tip;
 
-  /** @brief Called when button is clicked (activated) */
-  void (*clicked)(GtkButton *button, gpointer userdata);
+  /** @brief Associated menu item or NULL */
+  const char *menuitem;
 
   /** @brief Called to update button when state may have changed */
   void (*update)(const struct icon *i);
@@ -90,30 +93,114 @@ struct icon {
                 disorder_eclient_no_response *completed,
                 void *v);
 
+  /** @brief Flag */
+  unsigned flags;
+  
   /** @brief Pointer to button */
   GtkWidget *button;
+
+  /** @brief Pointer to menu item */
+  GtkWidget *item;
 };
 
+/** @brief This is the active half of a pair */
+#define ICON_ACTIVE 0x0001
+
+/** @brief This is the inactive half of a pair */
+#define ICON_INACTIVE 0x0002
+
 /** @brief Table of all icons */
 static struct icon icons[] = {
-  { "pause.png", "Pause playing track", clicked_icon, update_pause,
-    disorder_eclient_pause, 0 },
-  { "play.png", "Resume playing track", clicked_icon, update_play,
-    disorder_eclient_resume, 0 },
-  { "cross.png", "Cancel playing track", clicked_icon, update_scratch,
-    disorder_eclient_scratch_playing, 0 },
-  { "random.png", "Enable random play", clicked_icon, update_random_enable,
-    disorder_eclient_random_enable, 0 },
-  { "randomcross.png", "Disable random play", clicked_icon, update_random_disable,
-    disorder_eclient_random_disable, 0 },
-  { "notes.png", "Enable play", clicked_icon, update_enable,
-    disorder_eclient_enable, 0 },
-  { "notescross.png", "Disable play", clicked_icon, update_disable,
-    disorder_eclient_disable, 0 },
-  { "speaker.png", "Play network stream", clicked_icon, update_rtp,
-    enable_rtp, 0 },
-  { "speakercross.png", "Stop playing network stream", clicked_icon, update_nortp,
-    disable_rtp, 0 },
+  {
+    "pause.png",                        /* icon */
+    "Pause playing track",              /* tip */
+    "<GdisorderMain>/Control/Playing",  /* menuitem */
+    update_pause,                       /* update */
+    disorder_eclient_pause,             /* action */
+    ICON_ACTIVE,                        /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
+  {
+    "play.png",                         /* icon */
+    "Resume playing track",             /* tip */
+    "<GdisorderMain>/Control/Playing",  /* menuitem */
+    update_play,                        /* update */
+    disorder_eclient_resume,            /* action */
+    ICON_INACTIVE,                      /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
+  {
+    "cross.png",                        /* icon */
+    "Cancel playing track",             /* tip */
+    "<GdisorderMain>/Control/Scratch",  /* menuitem */
+    update_scratch,                     /* update */
+    disorder_eclient_scratch_playing,   /* action */
+    0,                                  /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
+  {
+    "random.png",                       /* icon */
+    "Enable random play",               /* tip */
+    "<GdisorderMain>/Control/Random play", /* menuitem */
+    update_random_enable,               /* update */
+    disorder_eclient_random_enable,     /* action */
+    ICON_INACTIVE,                      /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
+  {
+    "randomcross.png",                  /* icon */
+    "Disable random play",              /* tip */
+    "<GdisorderMain>/Control/Random play", /* menuitem */
+    update_random_disable,              /* update */
+    disorder_eclient_random_disable,    /* action */
+    ICON_ACTIVE,                        /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
+  {
+    "notes.png",                        /* icon */
+    "Enable play",                      /* tip */
+    0,                                  /* menuitem */
+    update_enable,                      /* update */
+    disorder_eclient_enable,            /* action */
+    ICON_INACTIVE,                      /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
+  {
+    "notescross.png",                   /* icon */
+    "Disable play",                     /* tip */
+    0,                                  /* menuitem */
+    update_disable,                     /* update */
+    disorder_eclient_disable,           /* action */
+    ICON_ACTIVE,                        /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
+  {
+    "speaker.png",                      /* icon */
+    "Play network stream",              /* tip */
+    "<GdisorderMain>/Control/Network player", /* menuitem */
+    update_rtp,                         /* update */
+    enable_rtp,                         /* action */
+    ICON_INACTIVE,                      /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
+  {
+    "speakercross.png",                 /* icon */
+    "Stop playing network stream",      /* tip */
+    "<GdisorderMain>/Control/Network player", /* menuitem */
+    update_nortp,                       /* update */
+    disable_rtp,                        /* action */
+    ICON_ACTIVE,                        /* flags */
+    0,                                  /* button */
+    0                                   /* item */
+  },
 };
 
 /** @brief Count of icons */
@@ -134,33 +221,40 @@ void control_monitor(void attribute((unused)) *u) {
 /** @brief Create the control bar */
 GtkWidget *control_widget(void) {
   GtkWidget *hbox = gtk_hbox_new(FALSE, 1), *vbox;
-  GtkWidget *content;
-  GdkPixbuf *pb;
   GtkWidget *v, *b;
   int n;
 
   NW(hbox);
   D(("control_widget"));
+  assert(mainmenufactory);              /* ordering must be right */
   for(n = 0; n < NICONS; ++n) {
     NW(button);
-    icons[n].button = gtk_button_new();
-    if((pb = find_image(icons[n].icon))) {
-      NW(image);
-      content = gtk_image_new_from_pixbuf(pb);
-    } else {
-      NW(label);
-      content = gtk_label_new(icons[n].icon);
-    }
-    gtk_container_add(GTK_CONTAINER(icons[n].button), content);
-    gtk_tooltips_set_tip(tips, icons[n].button, icons[n].tip, "");
+    icons[n].button = iconbutton(icons[n].icon, icons[n].tip);
     g_signal_connect(G_OBJECT(icons[n].button), "clicked",
-                     G_CALLBACK(icons[n].clicked), &icons[n]);
+                     G_CALLBACK(clicked_icon), &icons[n]);
     /* pop the icon in a vbox so it doesn't get vertically stretch if there are
      * taller things in the control bar */
     NW(vbox);
     vbox = gtk_vbox_new(FALSE, 0);
     gtk_box_pack_start(GTK_BOX(vbox), icons[n].button, TRUE, FALSE, 0);
     gtk_box_pack_start(GTK_BOX(hbox), vbox, FALSE, FALSE, 0);
+    if(icons[n].menuitem) {
+      icons[n].item = gtk_item_factory_get_widget(mainmenufactory,
+                                                  icons[n].menuitem);
+      switch(icons[n].flags & (ICON_ACTIVE|ICON_INACTIVE)) {
+      case ICON_ACTIVE:
+        g_signal_connect(G_OBJECT(icons[n].item), "toggled",
+                         G_CALLBACK(toggled_menu), &icons[n]);
+        break;
+      case ICON_INACTIVE:
+        /* Don't connect two instances of the signal! */
+        break;
+      default:
+        g_signal_connect(G_OBJECT(icons[n].item), "activate",
+                         G_CALLBACK(clicked_menu), &icons[n]);
+        break;
+      }
+    }
   }
   /* create the adjustments for the volume control */
   NW(adjustment);
@@ -175,6 +269,8 @@ GtkWidget *control_widget(void) {
   v = gtk_hscale_new(volume_adj);
   NW(hscale);
   b = gtk_hscale_new(balance_adj);
+  gtk_widget_set_style(v, tool_style);
+  gtk_widget_set_style(b, tool_style);
   gtk_scale_set_digits(GTK_SCALE(v), 10);
   gtk_scale_set_digits(GTK_SCALE(b), 10);
   gtk_widget_set_size_request(v, 192, -1);
@@ -207,10 +303,10 @@ void volume_update(void) {
   D(("volume_update"));
   l = volume_l / 100.0;
   r = volume_r / 100.0;
-  ++suppress_set_volume;
+  ++suppress_actions;
   gtk_adjustment_set_value(volume_adj, volume(l, r) * goesupto);
   gtk_adjustment_set_value(balance_adj, balance(l, r));
-  --suppress_set_volume;
+  --suppress_actions;
 }
 
 /** @brief Update the state of one of the control icons
@@ -234,6 +330,15 @@ static void update_icon(const struct icon *icon,
   /* Only both updating usability if the button is visible */
   if(visible)
     gtk_widget_set_sensitive(icon->button, usable);
+  if(icon->item) {
+    /* There's an associated menu item.  These are always visible, but may not
+     * be usable. */
+    if((icon->flags & (ICON_ACTIVE|ICON_INACTIVE)) == ICON_ACTIVE) {
+      /* The active half of a pair */
+      gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(icon->item), visible);
+    }
+    gtk_widget_set_sensitive(icon->item, usable);
+  }
 }
 
 static void update_pause(const struct icon *icon) {
@@ -294,7 +399,34 @@ static void clicked_icon(GtkButton attribute((unused)) *button,
                          gpointer userdata) {
   const struct icon *icon = userdata;
 
-  icon->action(client, 0, 0);
+  if(!suppress_actions)  
+    icon->action(client, 0, 0);
+}
+
+static void clicked_menu(GtkMenuItem attribute((unused)) *menuitem,
+                         gpointer userdata) {
+  const struct icon *icon = userdata;
+
+  if(!suppress_actions)
+    icon->action(client, 0, 0);
+}
+
+static void toggled_menu(GtkCheckMenuItem *menuitem,
+                         gpointer userdata) {
+  const struct icon *icon = userdata;
+  size_t n;
+
+  if(suppress_actions)
+    return;
+  /* This is a bit fiddlier than the others, we need to find the action for the
+   * new state.  If the new state is active then we want the ICON_INACTIVE
+   * version and vica versa. */
+  for(n = 0; n < NICONS; ++n)
+    if(icons[n].item == icon->item
+       && !!(icons[n].flags & ICON_INACTIVE) == !!menuitem->active)
+      break;
+  if(n < NICONS)
+    icons[n].action(client, 0, 0);
 }
 
 /** @brief Called when the volume has been adjusted */
@@ -303,7 +435,7 @@ static void volume_adjusted(GtkAdjustment attribute((unused)) *a,
   double v = gtk_adjustment_get_value(volume_adj) / goesupto;
   double b = gtk_adjustment_get_value(balance_adj);
 
-  if(suppress_set_volume)
+  if(suppress_actions)
     /* This is the result of an update from the server, not a change from the
      * user.  Don't feedback! */
     return;
@@ -313,10 +445,15 @@ static void volume_adjusted(GtkAdjustment attribute((unused)) *a,
   b = nearbyint(5 * b) / 5;
   /* Set the volume.  We don't want a reply, we'll get the actual new volume
    * from the log. */
-  disorder_eclient_volume(client, 0,
-                          nearbyint(left(v, b) * 100),
-                          nearbyint(right(v, b) * 100),
-                          0);
+  if(rtp_supported) {
+    int l = nearbyint(left(v, b) * 100), r = nearbyint(right(v, b) * 100);
+    mixer_control(&l, &r, 1);
+  } else
+    /* We don't want a reply, we'll get the actual new volume from the log. */
+    disorder_eclient_volume(client, 0,
+                            nearbyint(left(v, b) * 100),
+                            nearbyint(right(v, b) * 100),
+                            0);
 }
 
 /** @brief Formats the volume value */