]> gitweb.michael.orlitzky.com - xfce4-hdaps.git/blobdiff - panel-plugin/xfce4-hdaps.c
panel-plugin: replace XFCE_HVBOX with new GtkBox API.
[xfce4-hdaps.git] / panel-plugin / xfce4-hdaps.c
index 6f7bcfc4abc9910136a57fa3eead658987970039..e5a781e3b7cd50915f5a7136caeee63ad13d0c79 100644 (file)
@@ -1,7 +1,29 @@
+/*
+ * xfce4-hdaps, an XFCE4 panel plugin for the HDAPS system.
+ *
+ * Copyright (C) 2019 Michael Orlitzky
+ *
+ *   http://michael.orlitzky.com/
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details:
+ *
+ *   https://www.gnu.org/licenses/agpl-3.0.html
+ *
+ */
+
 #ifdef HAVE_CONFIG_H
 #include <config.h>
 #endif
 
+#include "hdaps.h"
 #include "xfce4-hdaps.h"
 #include "xfce4-hdaps-dialogs.h"
 
@@ -19,7 +41,7 @@
 static void hdaps_construct(XfcePanelPlugin *plugin);
 
 /* Register the plugin with the panel. */
-XFCE_PANEL_PLUGIN_REGISTER_EXTERNAL(hdaps_construct);
+XFCE_PANEL_PLUGIN_REGISTER(hdaps_construct);
 
 
 void hdaps_save(XfcePanelPlugin *plugin, HdapsPlugin *hdaps) {
@@ -69,10 +91,10 @@ void hdaps_set_icon(HdapsPlugin *hdaps, int status) {
   size = xfce_panel_plugin_get_size(hdaps->plugin) - 6;
 
   /* Try to load an icon from the current icon theme. */
-  if (status < HDAPS_OFF) {
+  if (status == HDAPS_ERROR) {
     /* Error */
     icon = gtk_icon_theme_load_icon(gtk_icon_theme_get_default(),
-                                   "dialog-error",
+                                   "emblem-noread",
                                    size, 0, NULL);
   }
   else if (status == HDAPS_OFF) {
@@ -107,10 +129,42 @@ void hdaps_set_icon(HdapsPlugin *hdaps, int status) {
 }
 
 
+
+void hdaps_set_tooltip(HdapsPlugin *hdaps, int status) {
+  if (status == HDAPS_ERROR) {
+    gtk_widget_set_tooltip_text(GTK_WIDGET(hdaps->eventbox), "HDAPS Error");
+  }
+  else if (status == HDAPS_OFF) {
+    gtk_widget_set_tooltip_text(GTK_WIDGET(hdaps->eventbox), "HDAPS Off");
+  }
+  else {
+    gtk_widget_set_tooltip_text(GTK_WIDGET(hdaps->eventbox), "HDAPS On");
+  }
+}
+
+
 static void hdaps_set_defaults(HdapsPlugin *hdaps) {
   DBG("Configuring all settings to defaults.");
-  hdaps->device_name = g_strdup(DEFAULT_DEVICE_NAME);
+
+  /* Here we determine the default device name. There are essentially
+     two "defaults," one soft, and the other hard. The soft default
+     is to choose the first supported HDAPS device in the system. This
+     would benefit users who, for example, only have one supported drive
+     named hda. If we can't find any supported HDAPS devices, we use the
+     hard default of DEFAULT_DEVICE_NAME. */
+  char hdaps_devices[MAX_HDAPS_DEVICES][FILENAME_MAX];
+  int found_devices = get_hdaps_device_list(hdaps_devices);
+
+  if (found_devices > 0) {
+    hdaps->device_name = g_strdup(hdaps_devices[0]);
+  }
+  else {
+    hdaps->device_name = g_strdup(DEFAULT_DEVICE_NAME);
+  }
+
   snprintf(hdaps->sysfs_file, FILENAME_MAX, UNLOAD_HEADS_FMT, hdaps->device_name);
+
+  /* The other default is easier. */
   hdaps->poll_frequency = DEFAULT_POLL_FREQUENCY;
 }
 
@@ -195,9 +249,14 @@ static HdapsPlugin *hdaps_new(XfcePanelPlugin *plugin) {
   hdaps->eventbox = gtk_event_box_new();
   gtk_widget_show(hdaps->eventbox);
 
+  /* Make the event box transparent. In newer versions of xfce4-panel
+     users can make the panel transparent, so we don't want to stick a
+     big opaque box on it. */
+  gtk_event_box_set_visible_window(GTK_EVENT_BOX(hdaps->eventbox), FALSE);
+
   /* Set up the hvbox for the widget, which supports
      both horizontal and vertical (hv) orientations. */
-  hdaps->hvbox = xfce_hvbox_new(orientation, FALSE, 2);
+  hdaps->hvbox = gtk_box_new(orientation, 2);
   gtk_widget_show(hdaps->hvbox);
   gtk_container_add(GTK_CONTAINER(hdaps->eventbox), hdaps->hvbox);
 
@@ -205,6 +264,7 @@ static HdapsPlugin *hdaps_new(XfcePanelPlugin *plugin) {
      so it's important that they start out n*sync. */
   hdaps->previous_status = HDAPS_OFF;
   hdaps_set_icon(hdaps, HDAPS_OFF);
+  hdaps_set_tooltip(hdaps, HDAPS_OFF);
   
   return hdaps;
 }
@@ -238,7 +298,7 @@ static void hdaps_free(XfcePanelPlugin *plugin,
   if (G_LIKELY(hdaps->device_name != NULL)) {
     g_free(hdaps->device_name);
   }
-  
+
   /* ...and finally free the plugin structure. */
   panel_slice_free(HdapsPlugin, hdaps);
 }
@@ -250,7 +310,7 @@ static void hdaps_orientation_changed(XfcePanelPlugin *plugin,
                                      HdapsPlugin     *hdaps) {
   
   /* Change the plugin's orientation. Basically magic to me. */
-  xfce_hvbox_set_orientation(XFCE_HVBOX(hdaps->hvbox), orientation);
+  gtk_orientable_set_orientation(GTK_ORIENTABLE(hdaps->hvbox), orientation);
 }
 
 
@@ -273,6 +333,9 @@ static gboolean hdaps_size_changed(XfcePanelPlugin *plugin,
     gtk_widget_set_size_request(GTK_WIDGET(plugin), size, -1);
   }
 
+  /* This fixes an issue where the initial icon size is too small. */
+  hdaps_set_icon(hdaps, hdaps->previous_status);
+
   /* We handled the change, so we're supposed to return TRUE. */
   return TRUE;
 }
@@ -286,9 +349,21 @@ static gboolean hdaps_update_status(HdapsPlugin *hdaps) {
   /* This just gets the status. */
   int status = parse_int_from_file(hdaps->sysfs_file);
 
+  /* The value in the sysfs_file represents the number of milliseconds
+   * that the drive heads will be parked. Of course, this will
+   * generally count down, and appear different each time we poll.
+   *
+   * So, to determine whether or not HDAPS has gone from "on"
+   * to "off," we treat all values greater than zero the same.
+  */
+  if (status > 0) {
+    status = HDAPS_ON;
+  }
+  
   if (status != hdaps->previous_status) {
     /* And we only update the icon if we need to. */
     hdaps_set_icon(hdaps, status);
+    hdaps_set_tooltip(hdaps, status);
     hdaps->previous_status = status;
   }