2006-04-06 [mones] 2.1.0cvs7
[claws.git] / src / prefs_themes.c
index c5d38a3e8978f17a1f09650b6162ce3c284c501e..292e45cc1eac955bf8ca0589996ad69879ec7802 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 2003-2004 Hiroyuki Yamamoto & the Sylpheed-Claws team
+ * Copyright (C) 2003-2006 Hiroyuki Yamamoto & the Sylpheed-Claws team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -139,6 +139,7 @@ static void prefs_themes_display_global_stats       (const ThemesData *tdata);
 static void prefs_themes_get_theme_info         (ThemesData *tdata);
 static void prefs_themes_display_theme_info     (ThemesData *tdata, const ThemeInfo *info);
 static void prefs_themes_get_themes_and_names  (ThemesData *tdata);
+static int prefs_themes_cmp_name(gconstpointer a, gconstpointer b);
 static void prefs_themes_free_names            (ThemesData *tdata);
 
 static void prefs_themes_set_themes_menu       (GtkOptionMenu *opmenu, const ThemesData *tdata);
@@ -185,8 +186,9 @@ static void prefs_themes_file_remove(const gchar *filename, gpointer data)
        
        base = g_path_get_basename(filename);
        if (TRUE == is_dir_exist(filename)) {
-               if (!((base[0] == '.') || (base[0] == '.' && base[1] == '.')))
-                       g_warning("prefs_themes_file_remove(): subdir in theme dir skipped.\n");
+               if (strcmp(base, ".") != 0 && strcmp(base, "..") != 0)
+                       g_warning("prefs_themes_file_remove(): subdir in theme dir skipped: '%s'.\n",
+                                               base);
        }
        else if (0 != g_unlink(filename)) {
                (*status) = g_strdup(filename);
@@ -204,8 +206,9 @@ static void prefs_themes_file_install(const gchar *filename, gpointer data)
        
        base = g_path_get_basename(filename);
        if (TRUE == is_dir_exist(filename)) {
-               if (!((base[0] == '.') || (base[0] == '.' && base[1] == '.')))
-                       g_warning("prefs_themes_file_install(): subdir in theme dir skipped.\n");
+               if (strcmp(base, ".") != 0 && strcmp(base, "..") !=0 )
+                       g_warning("prefs_themes_file_install(): subdir in theme dir skipped: '%s'.\n",
+                                               base);
        }
        else {
                gchar *fulldest;
@@ -265,16 +268,29 @@ static void prefs_themes_set_themes_menu(GtkOptionMenu *opmenu, const ThemesData
        GtkWidget *menu;
        GtkWidget *item;
        gint       i = 0, active = 0;
+       GList     *sorted_list = NULL;
 
        g_return_if_fail(opmenu != NULL);
-       
+
        gtk_option_menu_remove_menu(opmenu);
-       
+
+       /* sort theme data list by data name */
        menu = gtk_menu_new ();
+       while (themes != NULL) {
+               ThemeName *tname = (ThemeName *)(themes->data);
+
+               sorted_list = g_list_insert_sorted(sorted_list, (gpointer)(tname),
+                                                                               (GCompareFunc)prefs_themes_cmp_name);
+
+               themes = g_list_next(themes);
+       }
+
+       /* feed gtk_menu w/ sorted themes names */
+       themes = sorted_list;
        while (themes != NULL) {
                ThemeName *tname = (ThemeName *)(themes->data);
                gchar     *tpath = (gchar *)(tname->item->data);
-               
+
                item = gtk_menu_item_new_with_label(tname->name);
                gtk_widget_show(item);
                g_signal_connect(G_OBJECT(item), "activate",
@@ -284,15 +300,24 @@ static void prefs_themes_set_themes_menu(GtkOptionMenu *opmenu, const ThemesData
 
                if (tdata->displayed != NULL && tdata->displayed == tpath)
                        active = i;
+               ++i;
 
                themes = g_list_next(themes);
-               ++i;
        }
-       
+
+       g_list_free(sorted_list);
+
        gtk_menu_set_active(GTK_MENU(menu), active);
        gtk_option_menu_set_menu (opmenu, menu);
 }
 
+static int prefs_themes_cmp_name(gconstpointer a_p, gconstpointer b_p)
+{
+       /* compare two ThemeData structures by their name attribute */
+       return strcmp2((gchar *)(((ThemeName*)a_p)->name),
+                                       (gchar *)(((ThemeName*)b_p)->name));
+}
+
 static void prefs_themes_get_themes_and_names(ThemesData *tdata)
 {
        GList *tpaths;
@@ -337,7 +362,7 @@ void prefs_themes_init(void)
        path[1] = _("Themes");
        path[2] = NULL;
 
-       debug_print("Creating prefereces for themes...\n");
+       debug_print("Creating preferences for themes...\n");
        
        tdata = g_new0(ThemesData, 1);
        prefs_themes_data = tdata;
@@ -382,7 +407,7 @@ void prefs_themes_done(void)
 {
        ThemesData *tdata = prefs_themes_data;
 
-       debug_print("Finished prefereces for themes.\n");
+       debug_print("Finished preferences for themes.\n");
        
        stock_pixmap_themes_list_free(tdata->themes);
        prefs_themes_free_names(tdata); 
@@ -397,7 +422,6 @@ static void prefs_themes_btn_use_clicked_cb(GtkWidget *widget, gpointer data)
 
        theme_str = tdata->displayed;
        
-       if (prefs_common.pixmap_theme_path != NULL)
                g_free(prefs_common.pixmap_theme_path);
        
         prefs_common.pixmap_theme_path = g_strdup(theme_str);
@@ -435,10 +459,10 @@ static void prefs_themes_btn_remove_clicked_cb(GtkWidget *widget, gpointer data)
 
        val = alertpanel(alert_title,
                         _("Are you sure you want to remove this theme?"),
-                        GTK_STOCK_YES, GTK_STOCK_NO, NULL);
+                        GTK_STOCK_NO, GTK_STOCK_YES, NULL);
        g_free(alert_title);
 
-       if (G_ALERTDEFAULT == val) {
+       if (G_ALERTALTERNATE == val) {
                gchar *status = NULL;
                
                prefs_themes_foreach_file(theme_str, prefs_themes_file_remove, &status); 
@@ -490,21 +514,21 @@ static void prefs_themes_btn_install_clicked_cb(GtkWidget *widget, gpointer data
        if (file_exist(themeinfo, FALSE) == FALSE) {
                val = alertpanel(alert_title,
                                 _("This folder doesn't seem to be a theme folder.\nInstall anyway?"),
-                                GTK_STOCK_YES, GTK_STOCK_NO, NULL);
-               if (G_ALERTDEFAULT != val)
+                                GTK_STOCK_NO, GTK_STOCK_YES, NULL);
+               if (G_ALERTALTERNATE != val)
                        goto end_inst;
        }
        if (getuid() == 0) {
                val = alertpanel(alert_title,
                                 _("Do you want to install theme for all users?"),
-                                GTK_STOCK_YES, GTK_STOCK_NO, NULL);
+                                GTK_STOCK_NO, GTK_STOCK_YES, NULL);
                switch (val) {
-               case G_ALERTDEFAULT:
+               case G_ALERTALTERNATE:
                        cinfo->dest = g_strconcat(PACKAGE_DATA_DIR, G_DIR_SEPARATOR_S,
                                                  PIXMAP_THEME_DIR, G_DIR_SEPARATOR_S, 
                                                  themename, NULL);
                        break;
-               case G_ALERTALTERNATE:
+               case G_ALERTDEFAULT:
                        break;
                default:
                        goto end_inst;
@@ -546,7 +570,7 @@ static void prefs_themes_btn_install_clicked_cb(GtkWidget *widget, gpointer data
        else
                alertpanel_error(_("File %s failed\nwhile installing theme."), cinfo->status);
 end_inst:
-       if (cinfo->dest != NULL) g_free(cinfo->dest);
+       g_free(cinfo->dest);
        g_free(filename);
        g_free(source);
        g_free(themeinfo);
@@ -554,11 +578,6 @@ end_inst:
        g_free(themename);
 }
 
-static void prefs_themes_btn_more_clicked_cb(GtkWidget *widget, gpointer data)
-{
-       open_uri(CLAWS_THEMES_URI, prefs_common.uri_cmd);
-}
-
 static void prefs_themes_menu_item_activated_cb(GtkWidget *widget, gpointer data)
 {
        ThemesData *tdata = prefs_themes_data;
@@ -780,14 +799,6 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
 {
        ThemesPage *prefs_themes = (ThemesPage *)page;
        ThemesData *tdata = prefs_themes_data;
-       gchar *buf;
-       const gchar *tmp;
-       gint   i;
-       /* from gtk/about.c */
-       GtkStyle *style;
-       GdkColormap *cmap;
-       GdkColor uri_color[2] = {{0, 0, 0, 0xffff}, {0, 0xffff, 0, 0}};
-       gboolean success[2];
 
        GtkWidget *vbox1;
        GtkWidget *frame1;
@@ -823,16 +834,18 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
        GtkWidget *btn_use;
        GtkWidget *btn_remove;
 
-       vbox1 = gtk_vbox_new (FALSE, 0);
+       vbox1 = gtk_vbox_new (FALSE, VSPACING);
+       gtk_container_set_border_width (GTK_CONTAINER (vbox1), VBOX_BORDER);
        gtk_widget_show (vbox1);
 
        PACK_FRAME (vbox1, frame1, _("Selector"));
 
-       vbox2 = gtk_vbox_new (FALSE, 0);
+       vbox2 = gtk_vbox_new (FALSE, VSPACING);
+       gtk_container_set_border_width (GTK_CONTAINER (vbox2), VBOX_BORDER);
        gtk_widget_show (vbox2);
        gtk_container_add (GTK_CONTAINER (frame1), vbox2);
 
-       hbox3 = gtk_hbox_new (FALSE, 0);
+       hbox3 = gtk_hbox_new (FALSE, 5);
        gtk_widget_show (hbox3);
        gtk_box_pack_start (GTK_BOX (vbox2), hbox3, FALSE, FALSE, 0);
        gtk_container_set_border_width (GTK_CONTAINER (hbox3), 5);
@@ -851,30 +864,9 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
        gtk_box_pack_start (GTK_BOX (hbox3), btn_install, FALSE, FALSE, 0);
        GTK_WIDGET_SET_FLAGS (btn_install, GTK_CAN_DEFAULT);
 
-       btn_more = gtk_button_new_with_label (_("Get more..."));
+       btn_more = gtkut_get_link_btn(window, THEMES_URI, _("Get more..."));
        gtk_widget_show (btn_more);
        gtk_box_pack_start (GTK_BOX (hbox3), btn_more, FALSE, FALSE, 0);
-       GTK_WIDGET_SET_FLAGS (btn_more, GTK_CAN_DEFAULT);
-       /* make it look like an uri */
-       gtk_button_set_relief(GTK_BUTTON(btn_more), GTK_RELIEF_NONE);
-       tmp = gtk_label_get_text(GTK_LABEL(GTK_BIN(btn_more)->child));
-       buf = g_strdup(tmp);
-       for (i = 0; buf[i] != '\0'; buf[i++] = '_');
-       gtk_label_set_pattern(GTK_LABEL(GTK_BIN(btn_more)->child), buf);
-       g_free(buf);
-       cmap = gdk_drawable_get_colormap((mainwindow_get_mainwindow())->window->window);
-       gdk_colormap_alloc_colors(cmap, uri_color, 2, FALSE, TRUE, success);
-       if (success[0] == TRUE && success[1] == TRUE) {
-               gtk_widget_ensure_style(GTK_BIN(btn_more)->child);
-               style = gtk_style_copy
-                       (gtk_widget_get_style(GTK_BIN(btn_more)->child));
-               style->fg[GTK_STATE_NORMAL]   = uri_color[0];
-               style->fg[GTK_STATE_ACTIVE]   = uri_color[1];
-               style->fg[GTK_STATE_PRELIGHT] = uri_color[0];
-               gtk_widget_set_style(GTK_BIN(btn_more)->child, style);
-       } else
-               g_warning("prefs_themes_create_widget(): color allocation failed.\n");
-
 
        label_global_status = gtk_label_new ("");
        gtk_widget_show (label_global_status);
@@ -1017,9 +1009,6 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
        g_signal_connect(G_OBJECT(btn_install), "clicked",
                         G_CALLBACK(prefs_themes_btn_install_clicked_cb),
                         NULL);
-       g_signal_connect(G_OBJECT(btn_more), "clicked",
-                        G_CALLBACK(prefs_themes_btn_more_clicked_cb),
-                        NULL);
 
        prefs_themes->window = GTK_WIDGET(window);