[Goodies-commits] r6140 - in xfce4-timer-plugin/trunk: . src

Kemal Eroglu ilgar at xfce.org
Thu Nov 20 21:18:48 CET 2008


Author: ilgar
Date: 2008-11-20 20:18:48 +0000 (Thu, 20 Nov 2008)
New Revision: 6140

Modified:
   xfce4-timer-plugin/trunk/ChangeLog
   xfce4-timer-plugin/trunk/configure.in.in
   xfce4-timer-plugin/trunk/src/xfcetimer.c
Log:
Apply Fabian Nowak's changes

Modified: xfce4-timer-plugin/trunk/ChangeLog
===================================================================
--- xfce4-timer-plugin/trunk/ChangeLog	2008-11-20 20:16:39 UTC (rev 6139)
+++ xfce4-timer-plugin/trunk/ChangeLog	2008-11-20 20:18:48 UTC (rev 6140)
@@ -1,3 +1,7 @@
+20/11/2008 Kemal Ilgar Eroglu <ilgar_eroglu at yahoo.com>
+	* Version 0.6.1.
+	* Committed Fabian Nowak improvements on the UI.
+	* Xfce timer... now with German translations! :)
 25/12/2007 Kemal Ilgar Eroglu <ilgar_eroglu at yahoo.com>
 	* Version 0.6.
 	* Fix http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=456316 .

Modified: xfce4-timer-plugin/trunk/configure.in.in
===================================================================
--- xfce4-timer-plugin/trunk/configure.in.in	2008-11-20 20:16:39 UTC (rev 6139)
+++ xfce4-timer-plugin/trunk/configure.in.in	2008-11-20 20:18:48 UTC (rev 6140)
@@ -4,7 +4,7 @@
 dnl 	Please disable it in the Anjuta project configuration
 
 dnl version info
-m4_define([xfce4_timer_version], [0.6])
+m4_define([xfce4_timer_version], [0.6.1])
 
 dnl init autoconf
 AC_INIT([xfce4-timer-plugin], [xfce4_timer_version], [ilgar_eroglu at yahoo.com])

Modified: xfce4-timer-plugin/trunk/src/xfcetimer.c
===================================================================
--- xfce4-timer-plugin/trunk/src/xfcetimer.c	2008-11-20 20:16:39 UTC (rev 6139)
+++ xfce4-timer-plugin/trunk/src/xfcetimer.c	2008-11-20 20:18:48 UTC (rev 6140)
@@ -1,4 +1,4 @@
-/*  
+/*
  *
  *  Copyright (C) 2005 Kemal Ilgar Eroglu <kieroglu at math.washington.edu>
  *
@@ -18,9 +18,12 @@
  */
 
 
-#define TIMEOUT_TIME 	2000 /* Countdown update period in milliseconds */
-#define PBAR_THICKNESS 	10
+#define TIMEOUT_TIME    2000 /* Countdown update period in milliseconds */
+#define PBAR_THICKNESS  10
 
+#define BORDER 8
+#define WIDGET_SPACING 2
+
 #ifdef HAVE_CONFIG_H
 #include <config.h>
 #endif
@@ -45,27 +48,27 @@
 void make_menu(plugin_data *pd);
 
 /**
- * This is the timeout function that 
+ * This is the timeout function that
  * repeats the alarm.
 **/
 static gboolean repeat_alarm (gpointer data){
 
-  plugin_data *pd=(plugin_data *)data;  
-  
+  plugin_data *pd=(plugin_data *)data;
+
   /* Don't repeat anymore */
   if(pd->rem_repetitions == 0){
 
-	if(pd->timeout_command)
-		g_free(pd->timeout_command);
-	pd->timeout_command=NULL;   
-	pd->alarm_repeating=FALSE;
-	make_menu(pd);
-	
-  	return FALSE;
-  	
-  
+    if(pd->timeout_command)
+        g_free(pd->timeout_command);
+    pd->timeout_command=NULL;
+    pd->alarm_repeating=FALSE;
+    make_menu(pd);
+
+    return FALSE;
+
+
   }
-  
+
   g_spawn_command_line_async (pd->timeout_command,NULL);
   pd->rem_repetitions = pd->rem_repetitions -1 ;
   return TRUE;
@@ -73,7 +76,7 @@
 
 
 /**
- * This is the timeout function that updates the 
+ * This is the timeout function that updates the
  * tooltip, pbar and keeps track of elapsed time
 **/
 static gboolean timeout_function (gpointer data){
@@ -87,7 +90,7 @@
   elapsed_sec=(gint)g_timer_elapsed(pd->timer,&zip);
 
   /*g_fprintf(stderr,"\nElapsed %d seconds of %d",elapsed_sec,pd-
-					>timeout_period_in_sec);*/
+                    >timeout_period_in_sec);*/
 
   /* If countdown is not over, update tooltip */
   if(elapsed_sec < pd->timeout_period_in_sec){
@@ -95,34 +98,34 @@
      remaining=pd->timeout_period_in_sec-elapsed_sec;
 
      if(remaining>=3600)
-       g_snprintf(tiptext,31,_("%dh %dm %ds left"),remaining/3600, (remaining%3600)/60, 
-			remaining%60);
+       g_snprintf(tiptext,31,_("%dh %dm %ds left"),remaining/3600, (remaining%3600)/60,
+            remaining%60);
      else if (remaining>=60)
        g_snprintf(tiptext,31,_("%dm %ds left"),remaining/60, remaining%60);
      else
        g_snprintf(tiptext,31,_("%ds left"),remaining);
 
-     gtk_progress_bar_set_fraction	(GTK_PROGRESS_BAR(pd->pbar),
-					((gdouble)elapsed_sec)/pd->  						timeout_period_in_sec);  
+     gtk_progress_bar_set_fraction  (GTK_PROGRESS_BAR(pd->pbar),
+                    ((gdouble)elapsed_sec)/pd->                         timeout_period_in_sec);
 
      gtk_tooltips_set_tip(pd->tip,GTK_WIDGET(pd->base),tiptext,NULL);
 
      return TRUE;
-     
+
   }
 
   /* Countdown is over, stop timer and free resources */
 
   /*g_fprintf(stderr,"\nTimer command is ==> %s...",pd->timeout_command);*/
-      
+
   if( (strlen(pd->timeout_command)==0) || !pd->nowin_if_alarm ) {
     gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(pd->pbar),1);
-    dialog = gtk_message_dialog_new 	(NULL,
-                                  	GTK_DIALOG_DESTROY_WITH_PARENT,
-                                  	GTK_MESSAGE_WARNING,
-                                  	GTK_BUTTONS_CLOSE,
-                                  	_("Beeep! :) \nTime is up!"));
-  
+    dialog = gtk_message_dialog_new     (NULL,
+                                    GTK_DIALOG_DESTROY_WITH_PARENT,
+                                    GTK_MESSAGE_WARNING,
+                                    GTK_BUTTONS_CLOSE,
+                                    _("Beeep! :) \nTime is up!"));
+
     g_signal_connect_swapped (dialog, "response",
                            G_CALLBACK (gtk_widget_destroy),
                            dialog);
@@ -131,23 +134,23 @@
 
   if(strlen(pd->timeout_command)>0) {
 
-	g_spawn_command_line_async (pd->timeout_command,NULL);
-	
-  	if(pd->repeat_alarm){
-  		pd->alarm_repeating=TRUE;
-  		pd->rem_repetitions=pd->repetitions;
-  		if(pd->repeat_timeout!=0)
-  			g_source_remove(pd->repeat_timeout);
-		pd->repeat_timeout = g_timeout_add(pd->repeat_interval*1000,repeat_alarm,pd);
-	}
-	else
-	{
-		if(pd->timeout_command)
-			g_free(pd->timeout_command);
-		pd->timeout_command=NULL;
-	}
+    g_spawn_command_line_async (pd->timeout_command,NULL);
+
+    if(pd->repeat_alarm){
+        pd->alarm_repeating=TRUE;
+        pd->rem_repetitions=pd->repetitions;
+        if(pd->repeat_timeout!=0)
+            g_source_remove(pd->repeat_timeout);
+        pd->repeat_timeout = g_timeout_add(pd->repeat_interval*1000,repeat_alarm,pd);
+    }
+    else
+    {
+        if(pd->timeout_command)
+            g_free(pd->timeout_command);
+        pd->timeout_command=NULL;
+    }
   }
-  
+
   if(pd->timer)
      g_timer_destroy(pd->timer);
   pd->timer=NULL;
@@ -158,7 +161,7 @@
   pd->timer_on=FALSE;
 
   /* reset pbar */
-  gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(pd->pbar),0);  
+  gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(pd->pbar),0);
 
   make_menu(pd);
 
@@ -179,15 +182,15 @@
 
   row_count=0;
 
-  /* Find the index of the menuitem selected, save it in pd->selected. Not very 
+  /* Find the index of the menuitem selected, save it in pd->selected. Not very
      elegant, though */
   while (GTK_MENU_ITEM(menuitem)!=g_array_index(pd->menuarray,GtkMenuItem*,row_count) )
      row_count++;
-   
+
   pd->selected=row_count;
 
   /*g_fprintf(stderr,"\n Selecten menuitem is %d",row_count);*/
-  
+
 }
 
 /**
@@ -195,8 +198,8 @@
  * start/stop item is selected in the popup menu
 **/
 
-static void start_stop_selected (GtkWidget* menuitem, GdkEventButton* event, gpointer 
-										data){
+static void start_stop_selected (GtkWidget* menuitem, GdkEventButton* event, gpointer
+                                        data){
 
   plugin_data *pd=(plugin_data *)data;
   GtkTreeIter iter;
@@ -212,7 +215,7 @@
 
   /* If counting down, we stop the timer and free the resources */
   if(pd->timer_on){
-      
+
     /*g_fprintf(stderr,"\nTimer is running, shutting down...");*/
     if(pd->timer)
        g_timer_destroy(pd->timer);
@@ -228,7 +231,7 @@
 
     /* Disable tooltips, reset pbar */
     gtk_tooltips_disable(pd->tip);
-    gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(pd->pbar),0);  
+    gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(pd->pbar),0);
 
     /* update menu*/
     make_menu(pd);
@@ -243,7 +246,7 @@
 
   valid = gtk_tree_model_get_iter_first (GTK_TREE_MODEL(pd->list), &iter);
   row_count=0;
-  
+
   /* Empty timer list-> Nothing to do. pd->selected=0, though. */
   if(!valid)
     return;
@@ -254,8 +257,8 @@
       row_count++;
   }
 
-  gtk_tree_model_get 	(GTK_TREE_MODEL(pd->list), &iter, 2, &timerinfo, 3, 
-			&tout_command, 4, &is_cd ,5, &time, -1);
+  gtk_tree_model_get    (GTK_TREE_MODEL(pd->list), &iter, 2, &timerinfo, 3,
+            &tout_command, 4, &is_cd ,5, &time, -1);
 
   /* This will not be freed until the timeout is destroyed */
   pd->timeout_command=tout_command;
@@ -277,12 +280,12 @@
      /*g_fprintf(stderr,"%d \n",cur_s);*/
 
      timeout_period=time*60 - ((60*cur_h + cur_m)*60 + cur_s);
- 
+
      if(timeout_period <0)
         timeout_period+= 24*60*60;
- 
+
   }
-  /* Else 'time' already gives the countdown period in seconds */ 
+  /* Else 'time' already gives the countdown period in seconds */
   else
       timeout_period=time;
 
@@ -291,7 +294,7 @@
   /* start the timer */
   pd->timer=g_timer_new();
   pd->timer_on=TRUE;
-  
+
   /* update stuff */
   make_menu(pd);
 
@@ -310,21 +313,21 @@
  * Callback when "Stop the alarm" is selected in the popup menu
 **/
 
-static void stop_repeating_alarm (GtkWidget* menuitem, GdkEventButton* event, gpointer 
-										data){
+static void stop_repeating_alarm (GtkWidget* menuitem, GdkEventButton* event, gpointer
+                                        data){
    plugin_data *pd=(plugin_data *)data;
 
    g_source_remove(pd->repeat_timeout);
-   
+
    pd->alarm_repeating=FALSE;
 
    if(pd->timeout_command){
-	g_free(pd->timeout_command);
-	pd->timeout_command=NULL;   
+    g_free(pd->timeout_command);
+    pd->timeout_command=NULL;
    }
 
-   make_menu(pd);   
-   
+   make_menu(pd);
+
 }
 
 /**
@@ -335,8 +338,8 @@
     plugin_data *pd=(plugin_data *)data;
 
     /*g_fprintf(stderr,"\nReceived click on button %d",event->button);*/
- 
 
+
     if(!pd->menu){
       g_fprintf(stderr,"\nNo menu\n");
       return;
@@ -378,11 +381,11 @@
   row_count=0;
 
   while (valid){
-    
+
       /* Run through the list, read name and timer period info */
 
-      /*g_fprintf(stderr,"\nMaking menuitem %d while selected is %d",row_count,pd-> 
-								selected);*/
+      /*g_fprintf(stderr,"\nMaking menuitem %d while selected is %d",row_count,pd->
+                                selected);*/
       gtk_tree_model_get(GTK_TREE_MODEL(pd->list),&iter,1,&timername,2,&timerinfo,-1);
       g_snprintf(itemtext,255,"%s (%s)",timername,timerinfo);
       menuitem=gtk_radio_menu_item_new_with_label(group,itemtext);
@@ -390,13 +393,13 @@
       g_free(timername);
       g_free(timerinfo);
       group = gtk_radio_menu_item_get_group (GTK_RADIO_MENU_ITEM (menuitem));
-      g_signal_connect	(G_OBJECT(menuitem),"button_press_event", 
-			G_CALLBACK(timer_selected),pd);
+      g_signal_connect  (G_OBJECT(menuitem),"button_press_event",
+            G_CALLBACK(timer_selected),pd);
       /* The selected timer is always active */
       if(row_count==pd->selected)
         gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(menuitem),TRUE);
       /* others are disabled when timer or alarm is already running */
-      else if(pd->timer_on || pd->alarm_repeating) 
+      else if(pd->timer_on || pd->alarm_repeating)
         gtk_widget_set_sensitive(GTK_WIDGET(menuitem),FALSE);
 
       gtk_menu_shell_append(GTK_MENU_SHELL(pd->menu),menuitem);
@@ -405,7 +408,7 @@
 
       /* We add the address of menuitem to the array */
       g_array_append_val(pd->menuarray,menuitem);
-      
+
       valid = gtk_tree_model_iter_next (GTK_TREE_MODEL(pd->list), &iter);
       row_count++;
   }
@@ -418,29 +421,29 @@
   /* Start/stop menu item */
 
   if(!pd->alarm_repeating){
-	  if(pd->timer_on)
-	    menuitem=gtk_menu_item_new_with_label(_("Stop timer"));
-	  else 
-	    menuitem=gtk_menu_item_new_with_label(_("Start timer"));
+      if(pd->timer_on)
+        menuitem=gtk_menu_item_new_with_label(_("Stop timer"));
+      else
+        menuitem=gtk_menu_item_new_with_label(_("Start timer"));
 
-	  gtk_menu_shell_append	(GTK_MENU_SHELL(pd->menu),menuitem);
-	  g_signal_connect	(G_OBJECT(menuitem),"button_press_event",
-				G_CALLBACK(start_stop_selected),pd);
-	  gtk_widget_show(menuitem);
+      gtk_menu_shell_append (GTK_MENU_SHELL(pd->menu),menuitem);
+      g_signal_connect  (G_OBJECT(menuitem),"button_press_event",
+                G_CALLBACK(start_stop_selected),pd);
+      gtk_widget_show(menuitem);
   }
-  
+
   /* Stop repeating alarm if so */
   if(pd->alarm_repeating) {
-	menuitem=gtk_menu_item_new_with_label(_("Stop the alarm"));
+    menuitem=gtk_menu_item_new_with_label(_("Stop the alarm"));
 
-	gtk_menu_shell_append	(GTK_MENU_SHELL(pd->menu),menuitem);
-	g_signal_connect	(G_OBJECT(menuitem),"button_press_event",
-			G_CALLBACK(stop_repeating_alarm),pd);
-	gtk_widget_show(menuitem);
-  
-	gtk_widget_show(pd->menu);
+    gtk_menu_shell_append   (GTK_MENU_SHELL(pd->menu),menuitem);
+    g_signal_connect    (G_OBJECT(menuitem),"button_press_event",
+            G_CALLBACK(stop_repeating_alarm),pd);
+    gtk_widget_show(menuitem);
+
+    gtk_widget_show(pd->menu);
   }
-      
+
 }
 
 
@@ -456,13 +459,13 @@
 
   /* Add item to the list */
   gtk_list_store_append(adata->pd->list,&iter);
-    
+
   gtk_list_store_set(GTK_LIST_STORE(adata->pd->list),&iter,
-			0,adata->pd->count,
-			1,gtk_entry_get_text(GTK_ENTRY(adata->name)),
-			3,gtk_entry_get_text(GTK_ENTRY(adata->command)), 
-			4,gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(adata-> 
-									rb1)),-1);
+            0,adata->pd->count,
+            1,gtk_entry_get_text(GTK_ENTRY(adata->name)),
+            3,gtk_entry_get_text(GTK_ENTRY(adata->command)),
+            4,gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(adata->
+                                    rb1)),-1);
   /* Item count goes up by one */
   adata->pd->count=adata->pd->count+1;
 
@@ -496,7 +499,7 @@
 
   }
 
-  /* Update menu */  
+  /* Update menu */
   make_menu(adata->pd);
 
   /* Free resources */
@@ -513,7 +516,7 @@
 static void cancel_add_edit(GtkButton *button, gpointer data){
 
   alarm_data *adata=(alarm_data *)data;
- 
+
   gtk_widget_destroy(GTK_WIDGET(adata->window));
 
   g_free(adata);
@@ -537,12 +540,12 @@
   select = gtk_tree_view_get_selection (GTK_TREE_VIEW (adata->pd->tree));
 
   if (gtk_tree_selection_get_selected (select, &model, &iter))
-  {    
+  {
      gtk_list_store_set(GTK_LIST_STORE(adata->pd->list),&iter,
-			1,gtk_entry_get_text(GTK_ENTRY(adata->name)),
-			3,gtk_entry_get_text(GTK_ENTRY(adata->command)), 
-			4,gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(adata-> 
-									rb1)),-1);
+            1,gtk_entry_get_text(GTK_ENTRY(adata->name)),
+            3,gtk_entry_get_text(GTK_ENTRY(adata->command)),
+            4,gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(adata->
+                                    rb1)),-1);
      if(gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(adata->rb1))){
 
         t1=gtk_spin_button_get_value_as_int(GTK_SPIN_BUTTON(adata->timeh));
@@ -569,7 +572,7 @@
         gtk_list_store_set(GTK_LIST_STORE(adata->pd->list),&iter,2,timeinfo,-1);
 
       }
-  
+
   }
 
   make_menu(adata->pd);
@@ -579,8 +582,35 @@
   g_free(adata);
 }
 
+/**
+ * Callback when first radio button in dialog has been selected.
+ */
+static void alarmdialog_countdown_toggled (GtkButton *button, gpointer data)
+{
+  //plugin_data *pd = (plugin_data *)data;
+  alarm_data *adata = (alarm_data *) data;
+  gboolean active;
 
+  active = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(button));
+  gtk_widget_set_sensitive(GTK_WIDGET(adata->timeh), active);
+  gtk_widget_set_sensitive(GTK_WIDGET(adata->timem), active);
+  gtk_widget_set_sensitive(GTK_WIDGET(adata->times), active);
+}
+
 /**
+ * Callback when second radio button in dialog has been selected.
+ */
+static void alarmdialog_alarmtime_toggled (GtkButton *button, gpointer data)
+{
+  alarm_data *adata = (alarm_data *) data;
+  gboolean active;
+
+  active = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(button));
+  gtk_widget_set_sensitive(GTK_WIDGET(adata->time_h), active);
+  gtk_widget_set_sensitive(GTK_WIDGET(adata->time_m), active);
+}
+
+/**
  * Callback to the Add button in options window.
  * Creates the Add window.
 **/
@@ -604,98 +634,109 @@
 
   adata->window=window;
   adata->pd=pd;
-  
+
   gtk_window_set_modal(GTK_WINDOW(window),TRUE);
 
-  vbox=gtk_vbox_new(FALSE,0);
+  vbox=gtk_vbox_new(FALSE, BORDER);
   gtk_container_add(GTK_CONTAINER(window),vbox);
+  gtk_container_set_border_width(GTK_CONTAINER(window), BORDER);
 
   /***********/
-  hbox=gtk_hbox_new(TRUE,0);
-  gtk_box_pack_start(GTK_BOX(vbox),hbox,FALSE,FALSE,0);  
+  hbox=gtk_hbox_new(FALSE, BORDER);
+  gtk_box_pack_start(GTK_BOX(vbox),hbox,FALSE,FALSE, WIDGET_SPACING);
 
-  label = (GtkLabel *)gtk_label_new (_("Name"));
+  label = (GtkLabel *)gtk_label_new (_("Name:"));
   name = (GtkEntry *) gtk_entry_new_with_max_length(1023);
   adata->name=name;
 
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),TRUE,TRUE,0);  
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(name),TRUE,TRUE,0);  
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,0);
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(name),TRUE,TRUE,0);
 
   /**********/
   rb1=(GtkRadioButton *)gtk_radio_button_new_with_label(NULL,_("Enter the countdown time"));
+  g_signal_connect(G_OBJECT(rb1), "toggled", G_CALLBACK(alarmdialog_countdown_toggled), adata);
   rb2=(GtkRadioButton *)gtk_radio_button_new_with_label(gtk_radio_button_get_group
-					(rb1),_("Enter the time of alarm (24h format)"));
+                    (rb1),_("Enter the time of alarm (24h format)"));
+  g_signal_connect(G_OBJECT(rb2), "toggled", G_CALLBACK(alarmdialog_alarmtime_toggled), adata);
   adata->rb1=rb1;
 
   gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(rb1),TRUE,TRUE,0);
 
   hbox=gtk_hbox_new(FALSE,0);
-  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(hbox),TRUE,TRUE,0);  
+  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(hbox),TRUE,TRUE,0);
 
   timeh = (GtkSpinButton *)gtk_spin_button_new_with_range(0,23,1);
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(timeh),FALSE,FALSE,2);  
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(timeh),FALSE,FALSE, WIDGET_SPACING);
   adata->timeh=timeh;
   label = (GtkLabel *)gtk_label_new (_("h  "));
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,2);
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE, WIDGET_SPACING);
   timem = (GtkSpinButton *)gtk_spin_button_new_with_range(0,59,1);
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(timem),FALSE,FALSE,2);  
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(timem),FALSE,FALSE,WIDGET_SPACING);
   adata->timem=timem;
   label = (GtkLabel *)gtk_label_new (_("m  "));
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,2);
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,WIDGET_SPACING);
   times = (GtkSpinButton *)gtk_spin_button_new_with_range(0,59,1);
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(times),FALSE,FALSE,2);  
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(times),FALSE,FALSE,WIDGET_SPACING);
   adata->times=times;
   label = (GtkLabel *)gtk_label_new (_("s  "));
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,2);
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,WIDGET_SPACING);
 
-  label = (GtkLabel *)gtk_label_new (_("\nor\n"));
-  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(label),TRUE,TRUE,0);
+  label = (GtkLabel *)gtk_label_new (_("or"));
+  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(label),TRUE,TRUE,BORDER);
 
 
   gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(rb2),TRUE,TRUE,0);
 
-  hbox=gtk_hbox_new(FALSE,0);
-  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(hbox),TRUE,TRUE,0);  
+  hbox=gtk_hbox_new(FALSE,WIDGET_SPACING);
+  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(hbox),TRUE,TRUE,0);
 
   time_h = (GtkSpinButton *)gtk_spin_button_new_with_range(0,23,1);
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(time_h),FALSE,FALSE,0);  
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(time_h),FALSE,FALSE,0);
   adata->time_h=time_h;
   label = (GtkLabel *)gtk_label_new (":");
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,5);
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,WIDGET_SPACING);
   time_m = (GtkSpinButton *)gtk_spin_button_new_with_range(0,59,1);
-  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(time_m),FALSE,FALSE,5);  
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(time_m),FALSE,FALSE,WIDGET_SPACING);
   adata->time_m=time_m;
 
   /****************/
 
-  label = (GtkLabel *)gtk_label_new (_("\nThe command to run"));
-  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(label),TRUE,TRUE,0);
+  hbox=gtk_hbox_new(FALSE, BORDER);
+  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(hbox),TRUE,TRUE,WIDGET_SPACING);
+
+  label = (GtkLabel *)gtk_label_new (_("Command to run:"));
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(label),FALSE,FALSE,WIDGET_SPACING);
   command = (GtkEntry *)gtk_entry_new_with_max_length(1023);
   adata->command=command;
-  gtk_box_pack_start(GTK_BOX(vbox),GTK_WIDGET(command),TRUE,TRUE,0);
+  gtk_box_pack_start(GTK_BOX(hbox),GTK_WIDGET(command),TRUE,TRUE,WIDGET_SPACING);
 
   /****************/
 
-  hbox=gtk_hbox_new(TRUE,0);
-  gtk_box_pack_start(GTK_BOX(vbox),hbox,TRUE,TRUE,0);  
+  //hbox=gtk_hbox_new(TRUE,0);
+  hbox = gtk_hbutton_box_new();
+  gtk_button_box_set_layout(GTK_BUTTON_BOX(hbox), GTK_BUTTONBOX_END);
+  gtk_button_box_set_spacing(GTK_BUTTON_BOX(hbox), BORDER);
+  gtk_box_pack_start(GTK_BOX(vbox),hbox,TRUE,TRUE,WIDGET_SPACING);
+  //gtk_container_set_border_width(GTK_CONTAINER(hbox), BORDER);
 
   button=gtk_button_new_from_stock(GTK_STOCK_CANCEL);
   gtk_box_pack_start(GTK_BOX(hbox),button,TRUE,TRUE,0);
   g_signal_connect(G_OBJECT(button),"clicked",G_CALLBACK(cancel_add_edit),adata);
 
   button=gtk_button_new_from_stock(GTK_STOCK_OK);
-  gtk_box_pack_start(GTK_BOX(hbox),button,TRUE,TRUE,0);  
+  gtk_box_pack_start(GTK_BOX(hbox),button,TRUE,TRUE,0);
   if(GTK_WIDGET(buttonn)==pd->buttonadd)
      g_signal_connect(G_OBJECT(button),"clicked",G_CALLBACK(ok_add),adata);
   else
      g_signal_connect(G_OBJECT(button),"clicked",G_CALLBACK(ok_edit),adata);
 
-  
 
+
   /* If this is the add window, we're done */
   if(GTK_WIDGET(buttonn)==pd->buttonadd) {
     gtk_window_set_title(window,_("Add new alarm"));
     gtk_widget_show_all(GTK_WIDGET(window));
+    alarmdialog_alarmtime_toggled(GTK_BUTTON(rb2), adata);
     return;
   }
 
@@ -716,20 +757,20 @@
       gtk_tree_model_get(model,&iter,5,&time,-1);
 
       if(is_cd){
-         
+
          gtk_spin_button_set_value(GTK_SPIN_BUTTON(timeh),time/3600);
          gtk_spin_button_set_value(GTK_SPIN_BUTTON(timem),(time%3600)/60);
          gtk_spin_button_set_value(GTK_SPIN_BUTTON(times),time%60);
+         alarmdialog_alarmtime_toggled(GTK_BUTTON(rb2), adata);
          gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(rb1),TRUE);
       }
       else{
 
          gtk_spin_button_set_value(GTK_SPIN_BUTTON(time_h),time/60);
          gtk_spin_button_set_value(GTK_SPIN_BUTTON(time_m),time%60);
-         gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(rb2),TRUE);
-         
+         gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(rb2),TRUE); // active by default
       }
-     
+
   }
 
   gtk_window_set_title(window,_("Edit alarm"));
@@ -763,19 +804,19 @@
 
      /* Re-index the other rows */
 
-     /* The selected one is removed, the corresponding menuitem array is also 
+     /* The selected one is removed, the corresponding menuitem array is also
         updated */
      if(gtk_tree_selection_iter_is_selected(select,&iter)){
        /*g_fprintf(stderr,"\n Removing node %d ...\n", row_count);*/
        iter_remove=iter; /* Mark to be deleted */
        g_array_remove_index(pd->menuarray,row_count);
        if(pd->selected==row_count) /* Update the index of the selected item */
-          pd->selected=0; /* If the selected is deleted, new selected one is the first 
-				one. The first radiomenuitem gets activated anyway */
+          pd->selected=0; /* If the selected is deleted, new selected one is the first
+                one. The first radiomenuitem gets activated anyway */
        else if(pd->selected > row_count)
           pd->selected=pd->selected-1;  /* Those coming after are shifted one behind */
      }
-     else{  
+     else{
        /* Save new index on the remaning ones */
        gtk_list_store_set (pd->list, &iter, 0,row_count, -1);
        row_count++;
@@ -792,7 +833,7 @@
   pd->count=row_count;
 
   make_menu(pd);
-  
+
 }
 
 
@@ -817,19 +858,19 @@
   if(xfce_panel_plugin_get_orientation(plugin)==GTK_ORIENTATION_HORIZONTAL){
     pd->box=gtk_hbox_new(TRUE,0);
     gtk_container_add(GTK_CONTAINER(pd->eventbox),pd->box);
-    gtk_progress_bar_set_orientation	(GTK_PROGRESS_BAR(pd-> 
-					pbar),GTK_PROGRESS_BOTTOM_TO_TOP);
+    gtk_progress_bar_set_orientation    (GTK_PROGRESS_BAR(pd->
+                    pbar),GTK_PROGRESS_BOTTOM_TO_TOP);
     gtk_widget_set_size_request(GTK_WIDGET(pd->pbar),PBAR_THICKNESS,0);
     gtk_box_pack_start(GTK_BOX(pd->box),gtk_vseparator_new(),FALSE,FALSE,0);
     gtk_box_pack_start(GTK_BOX(pd->box),pd->pbar,FALSE,FALSE,0);
     gtk_box_pack_start(GTK_BOX(pd->box),gtk_vseparator_new(),FALSE,FALSE,0);
-    
+
   }
   else{ /* horizontal bar */
     pd->box=gtk_vbox_new(TRUE,0);
     gtk_container_add(GTK_CONTAINER(pd->eventbox),pd->box);
-    gtk_progress_bar_set_orientation	(GTK_PROGRESS_BAR(pd-> 
-	  				pbar),GTK_PROGRESS_LEFT_TO_RIGHT);
+    gtk_progress_bar_set_orientation    (GTK_PROGRESS_BAR(pd->
+                    pbar),GTK_PROGRESS_LEFT_TO_RIGHT);
     gtk_widget_set_size_request(GTK_WIDGET(pd->pbar),0,PBAR_THICKNESS);
     gtk_box_pack_start(GTK_BOX(pd->box),gtk_hseparator_new(),FALSE,FALSE,0);
     gtk_box_pack_start(GTK_BOX(pd->box),pd->pbar,FALSE,FALSE,0);
@@ -850,7 +891,7 @@
 
 /**
  * Loads the list from a keyfile, then saves them in a list_store
-**/ 
+**/
 static void load_settings(plugin_data *pd)
 {
 
@@ -864,82 +905,82 @@
   //if ((file=xfce_panel_plugin_lookup_rc_file (pd->base)) != NULL) {
 
 
-	  if (g_file_test(pd->configfile,G_FILE_TEST_EXISTS)) {  
-	  	
-		  rc = xfce_rc_simple_open (pd->configfile, TRUE);
+      if (g_file_test(pd->configfile,G_FILE_TEST_EXISTS)) {
 
-		  
-		  if (rc != NULL)
-		  {
+          rc = xfce_rc_simple_open (pd->configfile, TRUE);
 
 
-		  groupnum=0;
-		  g_sprintf(groupname,"G0");
+          if (rc != NULL)
+          {
 
 
-		  while(xfce_rc_has_group(rc,groupname)){
+          groupnum=0;
+          g_sprintf(groupname,"G0");
 
-				 xfce_rc_set_group(rc,groupname);
 
-			 /*g_fprintf(stderr,"\nLoading item %d\n",groupnum);*/
-			 gtk_list_store_append(pd->list,&iter);
+          while(xfce_rc_has_group(rc,groupname)){
 
-			 timerstring=(gchar *)xfce_rc_read_entry(rc,"timername","No name");
-			 gtk_list_store_set(pd->list,&iter,0,groupnum,1,timerstring,-1);
-			 /* g_free(timerstring); */ /* Entries read are not freed ! */
+                 xfce_rc_set_group(rc,groupname);
 
-			 timerstring=(gchar *)xfce_rc_read_entry(rc,"timercommand","");
-			 gtk_list_store_set(pd->list,&iter,3,timerstring,-1);
-			 /*g_fprintf(stderr,"\nLoaded timer command ==> %s... with length %d", 
-							timerstring,strlen(timerstring));*/
-			 /*g_free(timerstring);*/
+             /*g_fprintf(stderr,"\nLoading item %d\n",groupnum);*/
+             gtk_list_store_append(pd->list,&iter);
 
-			 timerstring=(gchar *)xfce_rc_read_entry(rc,"timerinfo","");
-			 gtk_list_store_set(pd->list,&iter,2,timerstring,-1);
+             timerstring=(gchar *)xfce_rc_read_entry(rc,"timername","No name");
+             gtk_list_store_set(pd->list,&iter,0,groupnum,1,timerstring,-1);
+             /* g_free(timerstring); */ /* Entries read are not freed ! */
 
-			 /*g_free(timerstring);*/
+             timerstring=(gchar *)xfce_rc_read_entry(rc,"timercommand","");
+             gtk_list_store_set(pd->list,&iter,3,timerstring,-1);
+             /*g_fprintf(stderr,"\nLoaded timer command ==> %s... with length %d",
+                            timerstring,strlen(timerstring));*/
+             /*g_free(timerstring);*/
 
-			 is_cd=xfce_rc_read_bool_entry(rc,"is_countdown",TRUE);
-			 time=xfce_rc_read_int_entry(rc,"time",0);
+             timerstring=(gchar *)xfce_rc_read_entry(rc,"timerinfo","");
+             gtk_list_store_set(pd->list,&iter,2,timerstring,-1);
 
-			 gtk_list_store_set(pd->list,&iter,4,is_cd,5,time,-1);
+             /*g_free(timerstring);*/
 
-			 groupnum++;
-			 g_snprintf(groupname,5,"G%d",groupnum);
-	   
-		  } /* end of while loop */
+             is_cd=xfce_rc_read_bool_entry(rc,"is_countdown",TRUE);
+             time=xfce_rc_read_int_entry(rc,"time",0);
 
-		  pd->count=groupnum;
-	  
+             gtk_list_store_set(pd->list,&iter,4,is_cd,5,time,-1);
 
-		  /* Read other options */
-		  if(xfce_rc_has_group(rc,"others")){
-			xfce_rc_set_group(rc,"others");
-			pd->nowin_if_alarm= xfce_rc_read_bool_entry
-				(rc,"nowin_if_alarm",FALSE);
-			pd->repeat_alarm= xfce_rc_read_bool_entry
-				(rc,"repeat_alarm",FALSE);
-			pd->repetitions= xfce_rc_read_int_entry
-				(rc,"repetitions",1);
-			pd->repeat_interval= xfce_rc_read_int_entry
-				(rc,"repeat_interval",10);
-					
-		  }	        
+             groupnum++;
+             g_snprintf(groupname,5,"G%d",groupnum);
 
-		  add_pbar(pd->base,pd);  
+          } /* end of while loop */
 
-			 xfce_rc_close(rc);
-		  }
-		  
-//	 }
-  }   
-  
+          pd->count=groupnum;
+
+
+          /* Read other options */
+          if(xfce_rc_has_group(rc,"others")){
+            xfce_rc_set_group(rc,"others");
+            pd->nowin_if_alarm= xfce_rc_read_bool_entry
+                (rc,"nowin_if_alarm",FALSE);
+            pd->repeat_alarm= xfce_rc_read_bool_entry
+                (rc,"repeat_alarm",FALSE);
+            pd->repetitions= xfce_rc_read_int_entry
+                (rc,"repetitions",1);
+            pd->repeat_interval= xfce_rc_read_int_entry
+                (rc,"repeat_interval",10);
+
+          }
+
+          add_pbar(pd->base,pd);
+
+             xfce_rc_close(rc);
+          }
+
+//   }
+  }
+
 }
 
 
 /**
  * Saves the list to a keyfile, backup a permanent copy
-**/ 
+**/
 static void save_settings(XfcePanelPlugin *plugin, plugin_data *pd){
 
   gchar *timername,*timercommand,*timerinfo;
@@ -955,10 +996,10 @@
   gsize size;
 
   FILE *conffile;
-  
+
   XfceRc *rc;
   gchar *file, *contents=NULL;
-  
+
   if (!(file = xfce_panel_plugin_save_location (plugin, TRUE)))
     return;
 
@@ -969,9 +1010,9 @@
   conffile = fopen(file,"w");
   if (conffile)
     fclose(conffile);
-    
+
   rc = xfce_rc_simple_open (file, FALSE);
-  
+
   if (!rc)
     return;
 
@@ -986,17 +1027,17 @@
   while (valid){
 
       gtk_tree_model_get (GTK_TREE_MODEL(pd->list), &iter,
-			1, &timername, 
-			2, &timerinfo, 
-			3, &timercommand, 
-			4, &is_cd,
-			5, &time, -1);
+            1, &timername,
+            2, &timerinfo,
+            3, &timercommand,
+            4, &is_cd,
+            5, &time, -1);
 
       g_snprintf(groupname,7,"G%d",row_count);
       xfce_rc_set_group(rc,groupname);
-      
+
       xfce_rc_write_entry(rc,"timername",timername);
-      
+
       xfce_rc_write_int_entry(rc,"time",time);
 
       xfce_rc_write_entry(rc,"timercommand",timercommand);
@@ -1014,21 +1055,21 @@
   }
 
 
-  /* save the other options */  
+  /* save the other options */
   xfce_rc_set_group(rc,"others");
 
   xfce_rc_write_bool_entry(rc,"nowin_if_alarm",pd->nowin_if_alarm);
-  xfce_rc_write_bool_entry(rc,"repeat_alarm",pd->repeat_alarm);  
-  xfce_rc_write_int_entry(rc,"repetitions",pd->repetitions);  
-  xfce_rc_write_int_entry(rc,"repeat_interval",pd->repeat_interval);  
+  xfce_rc_write_bool_entry(rc,"repeat_alarm",pd->repeat_alarm);
+  xfce_rc_write_int_entry(rc,"repetitions",pd->repetitions);
+  xfce_rc_write_int_entry(rc,"repeat_interval",pd->repeat_interval);
 
   xfce_rc_close(rc);
 
   conffile = fopen (pd->configfile,"w");
   /* We backup a permanent copy, which we'll use to load settings */
   if (conffile && g_file_get_contents(file,&contents,NULL,NULL)){
-  	fputs(contents,conffile);
-  	fclose(conffile);
+    fputs(contents,conffile);
+    fclose(conffile);
   }
 
   g_free(file);
@@ -1069,7 +1110,7 @@
   /* remove timeouts */
   if (pd->timeout!=0) g_source_remove(pd->timeout);
   if (pd->repeat_timeout!=0) g_source_remove(pd->repeat_timeout);
-  
+
   /*save_settings(pd);*/
 
   if(pd->timer)
@@ -1077,9 +1118,9 @@
 
   if(pd->timeout_command)
     g_free(pd->timeout_command);
-    
+
   if(pd->configfile)
-  	g_free(pd->configfile);
+    g_free(pd->configfile);
 
   gtk_object_destroy(GTK_OBJECT(pd->tip));
 
@@ -1099,7 +1140,7 @@
 
   /* free the plugin data structure */
   g_free(pd);
-  
+
   gtk_main_quit();
 }
 
@@ -1133,7 +1174,9 @@
   plugin_data *pd=(plugin_data *)data;
 
   pd->repeat_alarm = gtk_toggle_button_get_active(button);
-  
+  gtk_widget_set_sensitive(GTK_WIDGET(pd->spin_interval), pd->repeat_alarm);
+  gtk_widget_set_sensitive(GTK_WIDGET(pd->spin_repeat), pd->repeat_alarm);
+
   gtk_widget_set_sensitive(pd->repeat_alarm_box,pd->repeat_alarm);
 
 }
@@ -1179,61 +1222,62 @@
 
 
   xfce_panel_plugin_block_menu (plugin);
-    
-  dlg = gtk_dialog_new_with_buttons (_("Properties"), 
+
+  dlg = gtk_dialog_new_with_buttons (_("Xfce 4 Timer Plugin"),
               GTK_WINDOW (gtk_widget_get_toplevel (GTK_WIDGET (plugin))),
               GTK_DIALOG_DESTROY_WITH_PARENT |
               GTK_DIALOG_NO_SEPARATOR,
               GTK_STOCK_CLOSE, GTK_RESPONSE_OK,
               NULL);
-    
+
   g_signal_connect (dlg, "response", G_CALLBACK (options_dialog_response),
                     pd);
 
-  gtk_container_set_border_width (GTK_CONTAINER (dlg), 2);
-    
+  gtk_container_set_border_width (GTK_CONTAINER(dlg), BORDER);
+
   header = xfce_create_header (NULL, _("Xfce4 Timer Options"));
   gtk_widget_set_size_request (GTK_BIN (header)->child, 200, 32);
-  gtk_container_set_border_width (GTK_CONTAINER (header), 6);
+  //gtk_container_set_border_width (GTK_CONTAINER (header), 6);
   gtk_widget_show (header);
   gtk_box_pack_start (GTK_BOX (GTK_DIALOG (dlg)->vbox), header,
                       FALSE, TRUE, 0);
 
   gtk_box_pack_start (GTK_BOX (GTK_DIALOG (dlg)->vbox), vbox,
-                      FALSE, TRUE, 0);
-                        
+                      TRUE, TRUE, WIDGET_SPACING);
 
-  gtk_box_pack_start(GTK_BOX(vbox),hbox,TRUE,TRUE,0);
-  
+
+  hbox = gtk_hbox_new(FALSE, BORDER);
+  gtk_box_pack_start(GTK_BOX(vbox),hbox,TRUE,TRUE,BORDER);
+
   sw = gtk_scrolled_window_new (NULL, NULL);
 
   gtk_scrolled_window_set_shadow_type (GTK_SCROLLED_WINDOW (sw),
                                            GTK_SHADOW_ETCHED_IN);
   gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (sw),
-				 GTK_POLICY_AUTOMATIC,
-				 GTK_POLICY_AUTOMATIC);
-  
-  gtk_box_pack_start(GTK_BOX(hbox),sw,TRUE,TRUE,0);  
+                 GTK_POLICY_AUTOMATIC,
+                 GTK_POLICY_AUTOMATIC);
 
+  gtk_box_pack_start(GTK_BOX(hbox),sw,TRUE,TRUE,0);
+
   tree=gtk_tree_view_new_with_model(GTK_TREE_MODEL(pd->list));
   pd->tree=tree;
-  gtk_tree_view_set_rules_hint 	(GTK_TREE_VIEW (tree), TRUE);
-  gtk_tree_selection_set_mode 	(gtk_tree_view_get_selection (GTK_TREE_VIEW (tree)),
-				 GTK_SELECTION_SINGLE);
-  
+  gtk_tree_view_set_rules_hint  (GTK_TREE_VIEW (tree), TRUE);
+  gtk_tree_selection_set_mode   (gtk_tree_view_get_selection (GTK_TREE_VIEW (tree)),
+                 GTK_SELECTION_SINGLE);
+
   renderer = gtk_cell_renderer_text_new ();
 
-  column = gtk_tree_view_column_new_with_attributes (_("Timer\nname"), renderer,
-							"text", 1, NULL);
+  column = gtk_tree_view_column_new_with_attributes (_("Timer name"), renderer,
+                            "text", 1, NULL);
   gtk_tree_view_append_column (GTK_TREE_VIEW (tree), column);
 
 
-  column = gtk_tree_view_column_new_with_attributes (_("Countdown period /\nAlarm time"), 
-							renderer, "text", 2, NULL);
+  column = gtk_tree_view_column_new_with_attributes (_("Countdown period /\nAlarm time"),
+                            renderer, "text", 2, NULL);
   gtk_tree_view_append_column (GTK_TREE_VIEW (tree), column);
 
-  column = gtk_tree_view_column_new_with_attributes (_("Alarm command"), renderer, 
-							"text", 3, NULL);
+  column = gtk_tree_view_column_new_with_attributes (_("Alarm command"), renderer,
+                            "text", 3, NULL);
   gtk_tree_view_append_column (GTK_TREE_VIEW (tree), column);
 
 
@@ -1246,47 +1290,48 @@
 
   select = gtk_tree_view_get_selection (GTK_TREE_VIEW (pd->tree));
   gtk_tree_selection_set_mode (select, GTK_SELECTION_SINGLE);
-  g_signal_connect 	(G_OBJECT (select), "changed", 
-			G_CALLBACK(tree_selected), pd);
+  g_signal_connect  (G_OBJECT (select), "changed",
+            G_CALLBACK(tree_selected), pd);
 
 
   buttonbox=gtk_vbutton_box_new();
   gtk_button_box_set_layout(GTK_BUTTON_BOX(buttonbox),GTK_BUTTONBOX_START);
+  gtk_button_box_set_spacing(GTK_BUTTON_BOX(buttonbox),WIDGET_SPACING<<1);
   gtk_box_pack_start(GTK_BOX(hbox),buttonbox,FALSE,FALSE,0);
 
   button = gtk_button_new_from_stock (GTK_STOCK_ADD);
   pd->buttonadd=button;
-  gtk_box_pack_start(GTK_BOX (buttonbox), button, FALSE, FALSE,0);
+  gtk_box_pack_start(GTK_BOX (buttonbox), button, FALSE, FALSE,WIDGET_SPACING<<1);
   gtk_widget_set_sensitive(button,TRUE);
-  g_signal_connect (G_OBJECT (button), "clicked", G_CALLBACK(add_edit_clicked), 									pd);
+  g_signal_connect (G_OBJECT (button), "clicked", G_CALLBACK(add_edit_clicked),                                     pd);
 
 
   button = gtk_button_new_from_stock (GTK_STOCK_EDIT);
   pd->buttonedit=button;
-  gtk_box_pack_start(GTK_BOX (buttonbox), button, FALSE, FALSE,0);
+  gtk_box_pack_start(GTK_BOX (buttonbox), button, FALSE, FALSE,WIDGET_SPACING<<1);
   gtk_widget_set_sensitive(button,FALSE);
-  g_signal_connect (G_OBJECT (button), "clicked", G_CALLBACK(add_edit_clicked), 									pd);
+  g_signal_connect (G_OBJECT (button), "clicked", G_CALLBACK(add_edit_clicked),                                     pd);
 
   button = gtk_button_new_from_stock (GTK_STOCK_REMOVE);
   pd->buttonremove=button;
-  gtk_box_pack_start(GTK_BOX (buttonbox), button, FALSE, FALSE,0);
+  gtk_box_pack_start(GTK_BOX (buttonbox), button, FALSE, FALSE,WIDGET_SPACING);
   gtk_widget_set_sensitive(button,FALSE);
-  g_signal_connect (G_OBJECT (button), "clicked", G_CALLBACK(remove_clicked), 										pd);
+  g_signal_connect (G_OBJECT (button), "clicked", G_CALLBACK(remove_clicked),                                       pd);
 
-  gtk_box_pack_start(GTK_BOX(vbox),gtk_hseparator_new(),TRUE,TRUE,20);
+  gtk_box_pack_start(GTK_BOX(vbox),gtk_hseparator_new(),FALSE,FALSE,BORDER);
 
-  button=gtk_check_button_new_with_label(_("Don't display the warning window\nif an alarm command is set"));
+  button=gtk_check_button_new_with_label(_("Don't display a warning  if an alarm command is set"));
   gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(button),pd->nowin_if_alarm);
   g_signal_connect(G_OBJECT(button),"toggled",G_CALLBACK(toggle_nowin_if_alarm),pd);
-  gtk_box_pack_start(GTK_BOX(vbox),button,TRUE,TRUE,0);
+  gtk_box_pack_start(GTK_BOX(vbox),button,FALSE,FALSE,WIDGET_SPACING);
 
 
 
   button=gtk_check_button_new_with_label(_("Repeat the alarm command:"));
   gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(button),pd->repeat_alarm);
   g_signal_connect(G_OBJECT(button),"toggled",G_CALLBACK(toggle_repeat_alarm),pd);
-  gtk_box_pack_start(GTK_BOX(vbox),button,TRUE,TRUE,0);
-  
+  gtk_box_pack_start(GTK_BOX(vbox),button,FALSE,FALSE,WIDGET_SPACING);
+
   hbox = gtk_hbox_new (FALSE,0);
   pd->repeat_alarm_box=hbox;
   gtk_box_pack_start(GTK_BOX(hbox),gtk_label_new(_("Number of repetitions")),FALSE,FALSE,0);
@@ -1301,12 +1346,12 @@
   gtk_box_pack_start(GTK_BOX(hbox),spinbutton,FALSE,FALSE,0);
   gtk_spin_button_set_value(GTK_SPIN_BUTTON(spinbutton), pd->repeat_interval);
   g_signal_connect(G_OBJECT(spinbutton),"value-changed", G_CALLBACK(spin2_changed), pd);
-  
-  gtk_box_pack_start(GTK_BOX(vbox),hbox,TRUE,TRUE,0);
+
+  gtk_box_pack_start(GTK_BOX(vbox),hbox,FALSE,FALSE,WIDGET_SPACING);
   gtk_widget_set_sensitive(hbox,pd->repeat_alarm);
-  
 
 
+
   gtk_widget_show_all(GTK_WIDGET(dlg));
 
 }
@@ -1316,7 +1361,7 @@
  * create_sample_control
  *
  * Create a new instance of the plugin.
- * 
+ *
  * @control : #Control parent container
  *
  * Returns %TRUE on success, %FALSE on failure.
@@ -1327,7 +1372,7 @@
 
   GtkWidget *base,*menu,*socket,*menuitem,*box,*pbar2;
   GtkTooltips *tooltip;
-  char command[1024]; 
+  char command[1024];
   gchar *filename,*pathname;
 
 
@@ -1337,18 +1382,18 @@
   pd->base=plugin;
   pd->count=0;
   pd->pbar=gtk_progress_bar_new();
-  pd->list=gtk_list_store_new(6, 
-		 G_TYPE_INT,     /* Column 0: Index */
-		 G_TYPE_STRING,  /* Column 1: Name */
-		 G_TYPE_STRING,  /* Column 2: Timer period/alarm time */
-		 G_TYPE_STRING,  /* Command to run */
-		 G_TYPE_BOOLEAN, /* TRUE= Is countdown, i.e. h-m-s format. 
-				    FALSE= 24h format */
-		 G_TYPE_INT);    /* Timer period in seconds if countdown.
-				    Alarm time in minutes if 24h format is used,
-				    (i.e. 60 x Hr + Min) */
+  pd->list=gtk_list_store_new(6,
+         G_TYPE_INT,     /* Column 0: Index */
+         G_TYPE_STRING,  /* Column 1: Name */
+         G_TYPE_STRING,  /* Column 2: Timer period/alarm time */
+         G_TYPE_STRING,  /* Command to run */
+         G_TYPE_BOOLEAN, /* TRUE= Is countdown, i.e. h-m-s format.
+                    FALSE= 24h format */
+         G_TYPE_INT);    /* Timer period in seconds if countdown.
+                    Alarm time in minutes if 24h format is used,
+                    (i.e. 60 x Hr + Min) */
 
-  pd->eventbox=gtk_event_box_new();			 
+  pd->eventbox=gtk_event_box_new();
   pd->box=NULL;
   pd->timer_on=FALSE;
   pd->timeout=0;
@@ -1372,8 +1417,8 @@
 
   gtk_tooltips_set_tip(pd->tip, GTK_WIDGET(plugin), "", NULL);
   gtk_tooltips_disable(pd->tip);
- 
-  
+
+
   g_object_ref(pd->list);
 
   filename = xfce_panel_plugin_save_location (pd->base,TRUE);
@@ -1386,11 +1431,11 @@
 
   make_menu(pd);
 
-  g_signal_connect	(G_OBJECT(pd->eventbox), "button_press_event",
- 			G_CALLBACK(pbar_clicked), pd);
+  g_signal_connect  (G_OBJECT(pd->eventbox), "button_press_event",
+            G_CALLBACK(pbar_clicked), pd);
 
-  gtk_progress_bar_set_bar_style	(GTK_PROGRESS_BAR(pd->pbar), 
-					GTK_PROGRESS_CONTINUOUS);
+  gtk_progress_bar_set_bar_style    (GTK_PROGRESS_BAR(pd->pbar),
+                    GTK_PROGRESS_CONTINUOUS);
   gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(pd->pbar),0);
 
   add_pbar(pd->base,pd);
@@ -1399,22 +1444,22 @@
   /*gtk_widget_set_size_request(pd->eventbox,0,0);*/
   gtk_widget_set_size_request(GTK_WIDGET(plugin),10,10);
   xfce_panel_plugin_set_expand(plugin,FALSE);
-  
+
   gtk_container_add(GTK_CONTAINER(plugin),pd->eventbox);
 
   gtk_widget_show_all(GTK_WIDGET(plugin));
 
-  g_signal_connect (plugin, "free-data", 
+  g_signal_connect (plugin, "free-data",
                       G_CALLBACK (plugin_free), pd);
 
-  g_signal_connect (plugin, "save", 
+  g_signal_connect (plugin, "save",
                       G_CALLBACK (save_settings), pd);
 
-  g_signal_connect (plugin, "orientation-changed", 
+  g_signal_connect (plugin, "orientation-changed",
                       G_CALLBACK (orient_change), pd);
 
   xfce_panel_plugin_menu_show_configure (plugin);
-  g_signal_connect (plugin, "configure-plugin", 
+  g_signal_connect (plugin, "configure-plugin",
                       G_CALLBACK (plugin_create_options), pd);
   return(TRUE);
 }




More information about the Goodies-commits mailing list