Style the buttons correctly.
[tweakr] / modules / maemo-tweaks-mce.c
index e7013b9..4a83f86 100644 (file)
@@ -7,6 +7,7 @@
 #include <gtk/gtk.h>
 #include <hildon/hildon-picker-button.h>
 #include <hildon/hildon-touch-selector.h>
+#include <hildon/hildon-note.h>
 
 #include "libmaemo-tweaks-section/maemo-tweaks-section.h"
 #include "libmaemo-tweaks-section/maemo-tweaks-module.h"
         (G_TYPE_CHECK_INSTANCE_TYPE ((obj), \
         MAEMO_TWEAKS_TYPE_MCE_SECTION))
 
+#define DISABLE_LABEL "Disable"
+#define SHOW_MENU_LABEL "Show menu"
+#define POWER_OFF_LABEL "Power off"
+#define LOCK_LABEL "Lock"
+
 enum {
     SHORT_POWER_KEY_DISABLED,
     SHORT_POWER_KEY_MENU,
@@ -40,6 +46,14 @@ enum {
     LONG_POWER_KEY_N
 };
 
+enum {
+    DOUBLE_POWER_KEY_DISABLED,
+    DOUBLE_POWER_KEY_MENU,
+    DOUBLE_POWER_KEY_OFF,
+    DOUBLE_POWER_KEY_LOCK,
+    DOUBLE_POWER_KEY_N
+};
+
 
 typedef struct _picker_t
 {
@@ -49,16 +63,23 @@ typedef struct _picker_t
 } picker_t;
 
 static picker_t spk [] = {
-    {SHORT_POWER_KEY_DISABLED, "disabled", "Disabled"},
-    {SHORT_POWER_KEY_MENU, "menu", "Show menu"},
-    {SHORT_POWER_KEY_OFF, "poweroff", "Power off"}
+    {SHORT_POWER_KEY_DISABLED, "disabled", DISABLE_LABEL},
+    {SHORT_POWER_KEY_MENU, "menu", SHOW_MENU_LABEL},
+    {SHORT_POWER_KEY_OFF, "poweroff", POWER_OFF_LABEL}
 };
 
 static picker_t lpk [] = {
-    {SHORT_POWER_KEY_DISABLED, "disabled", "Disabled"},
-    {SHORT_POWER_KEY_MENU, "menu", "Show menu"},
-    {SHORT_POWER_KEY_OFF, "poweroff", "Power off"}
-};    
+    {LONG_POWER_KEY_DISABLED, "disabled", DISABLE_LABEL},
+    {LONG_POWER_KEY_MENU, "menu", SHOW_MENU_LABEL},
+    {LONG_POWER_KEY_OFF, "poweroff", POWER_OFF_LABEL}
+};
+
+static picker_t dpk [] = {
+    {DOUBLE_POWER_KEY_DISABLED, "disabled", DISABLE_LABEL},
+    {DOUBLE_POWER_KEY_MENU, "menu", SHOW_MENU_LABEL},
+    {DOUBLE_POWER_KEY_OFF, "poweroff", POWER_OFF_LABEL},
+    {DOUBLE_POWER_KEY_LOCK, "tklock", LOCK_LABEL}
+};
 
 
 typedef struct _MaemoTweaksMceSection MaemoTweaksMceSection;
@@ -72,6 +93,7 @@ struct _MaemoTweaksMceSection
     GKeyFile *ini;
     GtkWidget *short_power_key;
     GtkWidget *long_power_key;
+    GtkWidget *double_power_key;
 
     guint value_changed : 1;
 };
@@ -89,7 +111,8 @@ static void maemo_tweaks_mce_section_init
     (MaemoTweaksMceSection *section);
 static void maemo_tweaks_mce_section_dispose (GObject *obj);
 
-static void _save (MaemoTweaksSection *section, gboolean *requires_restart);
+static gboolean _save (MaemoTweaksSection *section,
+                       gboolean *requires_restart);
 
 static GType maemo_tweaks_mce_section_type = 0;
 static MaemoTweaksSectionClass *
@@ -172,6 +195,7 @@ GtkWidget * _build_short_power_key (MaemoTweaksMceSection *section)
 
     button = hildon_picker_button_new (HILDON_SIZE_AUTO,
                                        HILDON_BUTTON_ARRANGEMENT_VERTICAL);
+    hildon_gtk_widget_set_theme_size (button, HILDON_SIZE_FINGER_HEIGHT);
 
     hildon_button_set_title (HILDON_BUTTON (button),
                              "Power key: short press");
@@ -195,11 +219,12 @@ GtkWidget * _build_long_power_key (MaemoTweaksMceSection *section)
     for (i = 0; i < LONG_POWER_KEY_N; i++)
     {
         hildon_touch_selector_append_text (HILDON_TOUCH_SELECTOR (selector),
-                                           spk[i].label);
+                                           lpk[i].label);
     }
 
     button = hildon_picker_button_new (HILDON_SIZE_AUTO,
                                        HILDON_BUTTON_ARRANGEMENT_VERTICAL);
+    hildon_gtk_widget_set_theme_size (button, HILDON_SIZE_FINGER_HEIGHT);
 
     hildon_button_set_title (HILDON_BUTTON (button),
                              "Power key: long press");
@@ -214,16 +239,48 @@ GtkWidget * _build_long_power_key (MaemoTweaksMceSection *section)
     return button;
 }
 
+GtkWidget * _build_double_power_key (MaemoTweaksMceSection *section)
+{
+    gint i;
+    GtkWidget *button, *selector;
+
+    selector = hildon_touch_selector_new_text ();
+    for (i = 0; i < DOUBLE_POWER_KEY_N; i++)
+    {
+        hildon_touch_selector_append_text (HILDON_TOUCH_SELECTOR (selector),
+                                           dpk[i].label);
+    }
+
+    button = hildon_picker_button_new (HILDON_SIZE_AUTO,
+                                       HILDON_BUTTON_ARRANGEMENT_VERTICAL);
+    hildon_gtk_widget_set_theme_size (button, HILDON_SIZE_FINGER_HEIGHT);
+
+    hildon_button_set_title (HILDON_BUTTON (button),
+                             "Power key: double press");
+    gtk_button_set_alignment (GTK_BUTTON (button), 0.0f, 0.5f);
+    hildon_picker_button_set_selector (HILDON_PICKER_BUTTON (button),
+                                       HILDON_TOUCH_SELECTOR (selector));
+
+    g_signal_connect (G_OBJECT (button), "value-changed",
+                      G_CALLBACK (_value_changed), section);
+
+    gtk_widget_show (button);
+    return button;
+}
+
+
 static void
 maemo_tweaks_mce_section_init (MaemoTweaksMceSection *section)
 {
     MaemoTweaksSection *iface;
     gchar *short_power_key_value;
     gchar *long_power_key_value;
+    gchar *double_power_key_value;
     gint i;
 
     section->short_power_key = _build_short_power_key (section);
     section->long_power_key = _build_long_power_key (section);
+    section->double_power_key = _build_double_power_key (section);
 
     section->ini = g_key_file_new ();
 
@@ -240,6 +297,9 @@ maemo_tweaks_mce_section_init (MaemoTweaksMceSection *section)
     long_power_key_value = g_key_file_get_string (section->ini, "PowerKey",
                                                   "PowerKeyLongAction",
                                                   NULL);
+    double_power_key_value = g_key_file_get_string (section->ini, "PowerKey",
+                                                    "PowerKeyDoubleAction",
+                                                    NULL);
 
     for (i = 0; i < SHORT_POWER_KEY_N; i++)
     {
@@ -259,15 +319,26 @@ maemo_tweaks_mce_section_init (MaemoTweaksMceSection *section)
         }
     }
 
+    for (i = 0; i < DOUBLE_POWER_KEY_N; i++)
+    {
+        if (g_strcmp0 (double_power_key_value, dpk[i].value) == 0)
+        {
+            hildon_picker_button_set_active
+                (HILDON_PICKER_BUTTON (section->double_power_key), i);
+        }
+    }
+
     section->value_changed = FALSE;
 
     iface = MAEMO_TWEAKS_SECTION (section);
+    iface->name = "Mce";
     iface->widget = gtk_vbox_new (FALSE, 0);
     gtk_box_pack_start (GTK_BOX (iface->widget), section->short_power_key,
-                        TRUE, TRUE, 0);
+                        FALSE, FALSE, 0);
     gtk_box_pack_start (GTK_BOX (iface->widget), section->long_power_key,
-                        TRUE, TRUE, 0);
-
+                        FALSE, FALSE, 0);
+    gtk_box_pack_start (GTK_BOX (iface->widget), section->double_power_key,
+                        FALSE, FALSE, 0);
 }
 
 static void
@@ -284,15 +355,15 @@ maemo_tweaks_mce_section_dispose (GObject *obj)
         (obj);
 }
 
-static void _save (MaemoTweaksSection *section, gboolean *requires_restart)
+static gboolean _save (MaemoTweaksSection *section,
+                       gboolean *requires_restart)
 {
-    gchar *argv[4];
-    gint short_active, long_active;
+    gchar *argv[5];
+    gint short_active, long_active, double_active;
+    gint i;
 
     if (!MAEMO_TWEAKS_MCE_SECTION (section)->value_changed)
-        return;
-
-    *requires_restart = TRUE;
+        return TRUE;
 
     short_active = hildon_picker_button_get_active
         (HILDON_PICKER_BUTTON (MAEMO_TWEAKS_MCE_SECTION
@@ -302,17 +373,43 @@ static void _save (MaemoTweaksSection *section, gboolean *requires_restart)
         (HILDON_PICKER_BUTTON (MAEMO_TWEAKS_MCE_SECTION
                                (section)->long_power_key));
 
+    double_active = hildon_picker_button_get_active
+        (HILDON_PICKER_BUTTON (MAEMO_TWEAKS_MCE_SECTION
+                               (section)->double_power_key));
+
+    if (short_active  == SHORT_POWER_KEY_DISABLED &&
+        long_active   == LONG_POWER_KEY_DISABLED  &&
+        double_active == DOUBLE_POWER_KEY_DISABLED)
+    {
+        GtkWidget *note;
+        gint retcode;
+
+        note = hildon_note_new_confirmation
+            (NULL, "Setting all Power Key options to \"Disabled\" means "
+             "that the only way to turn the device off is the removal of "
+             "the battery. Do you want to continue?");
+        retcode = gtk_dialog_run (GTK_DIALOG (note));
+        gtk_widget_destroy (note);
+
+        if (retcode == GTK_RESPONSE_CANCEL)
+            return FALSE;
+    }
+
+    *requires_restart = TRUE;
+
     argv[0] = g_strdup ("/usr/bin/maemo-tweaks-mce-save");
     argv[1] = g_strdup_printf ("%s", spk[short_active].value);
     argv[2] = g_strdup_printf ("%s", lpk[long_active].value);
-    argv[3] = NULL;
+    argv[3] = g_strdup_printf ("%s", dpk[double_active].value);
+    argv[4] = NULL;
 
     g_spawn_sync ("/home/user", argv, NULL,
                   G_SPAWN_STDOUT_TO_DEV_NULL | G_SPAWN_STDERR_TO_DEV_NULL,
                   NULL, NULL, NULL, NULL, NULL, NULL);
 
-    g_free (argv[0]);
-    g_free (argv[1]);
-    g_free (argv[2]);
+    for (i = 0; i < (sizeof (argv) / sizeof (gchar *)) - 1; i++)
+        g_free (argv[i]);
+
+    return TRUE;
 }