Fix 'Apply' when changing scaling options only
[claws.git] / src / prefs_themes.c
index cb8fd6f56f5d3dc5206233491f71eb0de9aa574b..10f6baf9a76c29e283e3e5f0f3e11fe5dff07145 100644 (file)
@@ -1,10 +1,10 @@
 /*
- * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 2003-2006 Hiroyuki Yamamoto & the Sylpheed-Claws team
+ * Claws Mail -- a GTK+ based, lightweight, and fast e-mail client
+ * Copyright (C) 2003-2015 Hiroyuki Yamamoto & the Claws Mail 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
- * the Free Software Foundation; either version 2 of the License, or
+ * 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,
@@ -13,8 +13,7 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
 #ifdef HAVE_CONFIG_H
@@ -27,6 +26,9 @@
 #include <stdlib.h>
 #include <string.h>
 #include <sys/stat.h>
+#ifdef HAVE_SVG
+#include <math.h>
+#endif
 
 #include <glib.h>
 #include <glib/gi18n.h>
@@ -45,6 +47,7 @@
 #include "mainwindow.h"
 #include "compose.h"
 #include "alertpanel.h"
+#include "addr_compl.h"
 
 #define IS_CURRENT_THEME(path)  (strcmp(prefs_common.pixmap_theme_path, path) == 0)
 #define IS_INTERNAL_THEME(path) (strcmp(DEFAULT_PIXMAP_THEME, path) == 0)
@@ -67,16 +70,20 @@ typedef struct _ThemesPage
        GtkWidget *author;
        GtkWidget *url;
        GtkWidget *status;
-       
+
        GtkWidget *icons[PREVIEW_ICONS];
-       
-       GtkWidget *btn_use;
+
        GtkWidget *btn_remove;
 
-       GdkPixmap *pixmaps[PREVIEW_ICONS];
-       GdkBitmap *masks[PREVIEW_ICONS];
+       GdkPixbuf *pixbufs[PREVIEW_ICONS];
 
-       /* gchar     *theme_path; */
+#ifdef HAVE_SVG
+       GtkWidget *checkbtn_enable_alpha;
+       GtkWidget *checkbtn_enable_scaling;
+       GtkWidget *checkbtn_scaling_auto;
+       GtkWidget *label_scaling_ppi;
+       GtkWidget *spinbtn_scaling_ppi;
+#endif
 } ThemesPage;
 
 typedef struct _ThemeInfo
@@ -107,6 +114,9 @@ typedef struct _DirInfo {
        gint bytes;
        gint files;
        gint pixms;
+       /* extensions info */
+       const char **supported;
+       gint *length;
 } DirInfo;
 
 typedef struct _CopyInfo {
@@ -128,11 +138,15 @@ StockPixmap prefs_themes_icons[PREVIEW_ICONS] = {
 
 
 
-static void prefs_themes_btn_use_clicked_cb    (GtkWidget *widget, gpointer data);
 static void prefs_themes_btn_remove_clicked_cb (GtkWidget *widget, gpointer data);
 static void prefs_themes_btn_install_clicked_cb        (GtkWidget *widget, gpointer data);
-static void prefs_themes_btn_more_clicked_cb   (GtkWidget *widget, gpointer data);
 static void prefs_themes_menu_item_activated_cb        (GtkWidget *widget, gpointer data);
+#ifdef HAVE_SVG
+static gdouble prefs_themes_compute_ppi(GdkScreen *screen);
+static gdouble prefs_themes_get_adjusted_ppi(void);
+static void prefs_themes_checkbtn_enable_scaling_toggled_cb (GtkWidget *widget, gpointer data);
+static void prefs_themes_checkbtn_scaling_auto_toggled_cb (GtkWidget *widget, gpointer data);
+#endif
 
 static void prefs_themes_update_buttons                (const ThemesData *tdata);
 static void prefs_themes_display_global_stats  (const ThemesData *tdata);
@@ -142,7 +156,7 @@ 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);
+static void prefs_themes_set_themes_menu       (GtkComboBox *combo, const ThemesData *tdata);
 
 static gchar *prefs_themes_get_theme_stats     (const gchar *dirname);
 static gboolean prefs_themes_is_system_theme   (const gchar *dirname);
@@ -160,18 +174,24 @@ static void prefs_themes_file_install             (const gchar *filename, gpointer data);
 
 static void prefs_themes_file_stats(const gchar *filename, gpointer data)
 {
-       struct stat s;
-       DirInfo    *di = (DirInfo *)data;
-       gint        len;
-       
-       if (0 == stat(filename, &s) && 0 != S_ISREG(s.st_mode)) {
+       GStatBuf s;
+       DirInfo *di = (DirInfo *)data;
+       gint len;
+       gint i;
+
+       if (0 == g_stat(filename, &s) && 0 != S_ISREG(s.st_mode)) {
                di->bytes += s.st_size;
                di->files++;
                len = strlen(filename);
-               if (len > 4) {
-                       if (filename[len - 1] == 'm' && filename[len - 2] == 'p' &&
-                           filename[len - 3] == 'x' && filename[len - 4] == '.')
+               for (i = 0; (di->supported)[i] != NULL; ++i) {
+                       gint curlen = (di->length)[i];
+                       if (len <= curlen)
+                               continue;
+                       const gchar *extension = filename + (len - curlen);
+                       if (!strcmp(extension, (di->supported)[i])) {
                                di->pixms++;
+                               break;
+                       }
                }
        }
 }
@@ -180,17 +200,17 @@ static void prefs_themes_file_remove(const gchar *filename, gpointer data)
 {
        gchar **status = (gchar **)data;
        gchar *base;
-       
+
        if ((*status) != NULL)
                return;
-       
+
        base = g_path_get_basename(filename);
        if (TRUE == is_dir_exist(filename)) {
                if (strcmp(base, ".") != 0 && strcmp(base, "..") != 0)
-                       g_warning("prefs_themes_file_remove(): subdir in theme dir skipped: '%s'.\n",
+                       g_warning("prefs_themes_file_remove(): subdir in theme dir skipped: '%s'.",
                                                base);
        }
-       else if (0 != g_unlink(filename)) {
+       else if (0 != claws_unlink(filename)) {
                (*status) = g_strdup(filename);
        }
        g_free(base);
@@ -200,21 +220,21 @@ static void prefs_themes_file_install(const gchar *filename, gpointer data)
 {
        CopyInfo *ci = (CopyInfo *)data;
        gchar *base;
-       
+
        if (ci->status != NULL)
                return;
-       
+
        base = g_path_get_basename(filename);
        if (TRUE == is_dir_exist(filename)) {
                if (strcmp(base, ".") != 0 && strcmp(base, "..") !=0 )
-                       g_warning("prefs_themes_file_install(): subdir in theme dir skipped: '%s'.\n",
+                       g_warning("prefs_themes_file_install(): subdir in theme dir skipped: '%s'.",
                                                base);
        }
        else {
                gchar *fulldest;
-               
+
                fulldest = g_strconcat(ci->dest, G_DIR_SEPARATOR_S, base, NULL);
-               
+
                if (0 != copy_file(filename, fulldest, FALSE)) {
                        ci->status = g_strdup(filename);
                }
@@ -225,80 +245,84 @@ static void prefs_themes_file_install(const gchar *filename, gpointer data)
 
 static void prefs_themes_foreach_file(const gchar *dirname, const FileFunc func, gpointer data)
 {
-       struct dirent *d;
-       DIR           *dp;
-
-       g_return_if_fail(dirname != NULL);
-       g_return_if_fail(func != NULL);
-       
-       if ((dp = opendir(dirname)) == NULL) {
-               debug_print("directory %s not found", dirname);
+       const gchar *entry;
+       gchar *fullentry;
+       GDir *dp;
+       GError *error = NULL;
+
+       cm_return_if_fail(dirname != NULL);
+       cm_return_if_fail(func != NULL);
+
+       if ((dp = g_dir_open(dirname, 0, &error)) == NULL) {
+               debug_print("couldn't open dir '%s': %s (%d)\n", dirname,
+                               error->message, error->code);
+               g_error_free(error);
                return;
        }
 
-       while ((d = readdir(dp)) != NULL) {
-               gchar *entry;
-               gchar *fullentry;
+       while ((entry = g_dir_read_name(dp)) != NULL) {
 
-               entry     = d->d_name;
                fullentry = g_strconcat(dirname, G_DIR_SEPARATOR_S, entry, NULL);
 
                (*func)(fullentry, data);
-               
+
                g_free(fullentry);
-       }       
+       }
+       g_dir_close(dp);
 }
 
 static gboolean prefs_themes_is_system_theme(const gchar *dirname)
 {
        gint len;
-       
-       g_return_val_if_fail(dirname != NULL, FALSE);
+       gchar *system_theme_dir;
+       gboolean is_sys = FALSE;
 
-       len = strlen(PACKAGE_DATA_DIR);
-       if (strlen(dirname) > len && 0 == strncmp(dirname, PACKAGE_DATA_DIR, len))
-               return TRUE;
-       
-       return FALSE;
+       cm_return_val_if_fail(dirname != NULL, FALSE);
+
+       system_theme_dir = stock_pixmap_get_system_theme_dir_for_theme(NULL);
+       len = strlen(system_theme_dir);
+       if (strlen(dirname) > len && 0 == strncmp(dirname, system_theme_dir, len))
+               is_sys = TRUE;
+
+       g_free(system_theme_dir);
+
+       return is_sys;
 }
 
-static void prefs_themes_set_themes_menu(GtkOptionMenu *opmenu, const ThemesData *tdata)
+static void prefs_themes_set_themes_menu(GtkComboBox *combo, const ThemesData *tdata)
 {
+       GtkListStore *store;
+       GtkTreeIter iter;
        GList     *themes = tdata->names;
-       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);
+       cm_return_if_fail(combo != NULL);
 
        /* 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);
+                                                  (GCompareFunc)prefs_themes_cmp_name);
 
                themes = g_list_next(themes);
        }
 
+       store = gtk_list_store_new(2, G_TYPE_STRING, G_TYPE_POINTER);
+
        /* 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",
-                                G_CALLBACK(prefs_themes_menu_item_activated_cb),
-                                tname->item->data);
-               gtk_menu_append(GTK_MENU(menu), item);
+               gtk_list_store_append(store, &iter);
+               gtk_list_store_set(store, &iter,
+                                  0, tname->name,
+                                  1, tname->item->data, -1);
 
-               if (tdata->displayed != NULL && tdata->displayed == tpath)
+               if (tdata->displayed != NULL && !strcmp2(tdata->displayed,tpath))
                        active = i;
                ++i;
 
@@ -307,8 +331,12 @@ static void prefs_themes_set_themes_menu(GtkOptionMenu *opmenu, const ThemesData
 
        g_list_free(sorted_list);
 
-       gtk_menu_set_active(GTK_MENU(menu), active);
-       gtk_option_menu_set_menu (opmenu, menu);
+       g_signal_connect(G_OBJECT(combo), "changed",
+                        G_CALLBACK(prefs_themes_menu_item_activated_cb),
+                        NULL);
+
+       gtk_combo_box_set_model(combo, GTK_TREE_MODEL(store));
+       gtk_combo_box_set_active(combo, active);
 }
 
 static int prefs_themes_cmp_name(gconstpointer a_p, gconstpointer b_p)
@@ -321,27 +349,27 @@ static int prefs_themes_cmp_name(gconstpointer a_p, gconstpointer b_p)
 static void prefs_themes_get_themes_and_names(ThemesData *tdata)
 {
        GList *tpaths;
-       
-       g_return_if_fail(tdata != NULL);
-       
+
+       cm_return_if_fail(tdata != NULL);
+
        if (tdata->themes != NULL)
                stock_pixmap_themes_list_free(tdata->themes);
        if (tdata->names != NULL)
                prefs_themes_free_names(tdata);
-       
+
        tdata->themes = stock_pixmap_themes_list_new();
-       
+
        tpaths = tdata->themes;
        while (tpaths != NULL) {
                ThemeName *name = g_new0(ThemeName, 1);
                gchar *sname = g_path_get_basename((const gchar *)(tpaths->data));
-               
+
                if (IS_INTERNAL_THEME(sname))
                        name->name = g_strdup(_("Default internal theme"));
                else
                        name->name = g_strdup(sname);
                name->item = tpaths;
-                       
+
                tdata->names = g_list_append(tdata->names, name);
                if (!strcmp2(tpaths->data, prefs_common.pixmap_theme_path)) {
                        tdata->displayed = (gchar *)tpaths->data;
@@ -363,14 +391,14 @@ void prefs_themes_init(void)
        path[2] = NULL;
 
        debug_print("Creating preferences for themes...\n");
-       
+
        tdata = g_new0(ThemesData, 1);
        prefs_themes_data = tdata;
 
        prefs_themes_get_themes_and_names(tdata);
-       
+
        page = g_new0(ThemesPage, 1);
-       
+
        page->page.path = path;
        page->page.create_widget = prefs_themes_create_widget;
        page->page.destroy_widget = prefs_themes_destroy_widget;
@@ -383,23 +411,27 @@ void prefs_themes_init(void)
        tpaths = g_list_first(tdata->themes);
        if (tdata->displayed == NULL)
                tdata->displayed = (gchar *)(tpaths->data);
+#ifdef HAVE_SVG
+       if (prefs_common.pixmap_scaling_auto)
+               prefs_common.pixmap_scaling_ppi = prefs_themes_get_adjusted_ppi();
+#endif
 }
 
 static void prefs_themes_free_names(ThemesData *tdata)
 {
        GList *names;
-       
+
        names = tdata->names;
        while (names != NULL) {
                ThemeName *tn = (ThemeName *)(names->data);
-               
+
                tn->item = NULL;
                g_free(tn->name);
                g_free(tn);
-               
+
                names = g_list_next(names);
        }
-       g_list_free(names);
+       g_list_free(tdata->names);
        tdata->names = NULL;
 }
 
@@ -408,31 +440,13 @@ void prefs_themes_done(void)
        ThemesData *tdata = prefs_themes_data;
 
        debug_print("Finished preferences for themes.\n");
-       
+
        stock_pixmap_themes_list_free(tdata->themes);
-       prefs_themes_free_names(tdata); 
+       prefs_themes_free_names(tdata); 
        g_free(tdata->page);
        g_free(tdata);
 }
 
-static void prefs_themes_btn_use_clicked_cb(GtkWidget *widget, gpointer data)
-{
-       ThemesData *tdata = prefs_themes_data;
-       gchar      *theme_str;
-
-       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);
-       
-        main_window_reflect_prefs_all_real(TRUE);
-        compose_reflect_prefs_pixmap_theme();
-       
-       prefs_themes_update_buttons(tdata);
-}
-
 static void prefs_themes_btn_remove_clicked_cb(GtkWidget *widget, gpointer data)
 {
        ThemesData *tdata = prefs_themes_data;
@@ -442,11 +456,11 @@ static void prefs_themes_btn_remove_clicked_cb(GtkWidget *widget, gpointer data)
        gchar      *tmp = NULL;
 
        theme_str = tdata->displayed;
-       
+
        tmp = g_path_get_basename(theme_str);
 
        if (IS_SYSTEM_THEME(theme_str)) {
-               if (getuid() != 0) {
+               if (!superuser_p()) {
                        alertpanel_error(_("Only root can remove system themes"));
                        return;
                }
@@ -465,7 +479,7 @@ static void prefs_themes_btn_remove_clicked_cb(GtkWidget *widget, gpointer data)
 
        if (G_ALERTALTERNATE == val) {
                gchar *status = NULL;
-               
+
                prefs_themes_foreach_file(theme_str, prefs_themes_file_remove, &status); 
                if (0 != rmdir(theme_str)) {
                        if (status != NULL) {
@@ -476,10 +490,10 @@ static void prefs_themes_btn_remove_clicked_cb(GtkWidget *widget, gpointer data)
                                alertpanel_error(_("Removing theme directory failed."));
                }
                else {  
-                       alertpanel_notice(_("Theme removed succesfully"));
+                       alertpanel_notice(_("Theme removed successfully"));
                        /* update interface back to first theme */
                        prefs_themes_get_themes_and_names(tdata);
-                       prefs_themes_set_themes_menu(GTK_OPTION_MENU(tdata->page->op_menu), tdata);
+                       prefs_themes_set_themes_menu(GTK_COMBO_BOX(tdata->page->op_menu), tdata);
                        prefs_themes_display_global_stats(tdata);
                        tdata->displayed = (gchar *)((g_list_first(tdata->themes))->data);
                        prefs_themes_get_theme_info(tdata);
@@ -495,11 +509,11 @@ static void prefs_themes_btn_install_clicked_cb(GtkWidget *widget, gpointer data
        CopyInfo   *cinfo;
        AlertValue  val = 0;
        ThemesData *tdata = prefs_themes_data;
-       
+
        filename = filesel_select_file_open_folder(_("Select theme folder"), NULL);
        if (filename == NULL) 
                return;
-       
+
        if (filename[strlen(filename) - 1] != G_DIR_SEPARATOR)
                filename = g_strconcat(filename, G_DIR_SEPARATOR_S, NULL);
        else
@@ -519,15 +533,14 @@ static void prefs_themes_btn_install_clicked_cb(GtkWidget *widget, gpointer data
                if (G_ALERTALTERNATE != val)
                        goto end_inst;
        }
-       if (getuid() == 0) {
+       if (superuser_p ()) {
                val = alertpanel(alert_title,
                                 _("Do you want to install theme for all users?"),
                                 GTK_STOCK_NO, GTK_STOCK_YES, NULL);
                switch (val) {
                case G_ALERTALTERNATE:
-                       cinfo->dest = g_strconcat(PACKAGE_DATA_DIR, G_DIR_SEPARATOR_S,
-                                                 PIXMAP_THEME_DIR, G_DIR_SEPARATOR_S, 
-                                                 themename, NULL);
+                       cinfo->dest = stock_pixmap_get_system_theme_dir_for_theme(
+                                               themename);
                        break;
                case G_ALERTDEFAULT:
                        break;
@@ -537,16 +550,27 @@ static void prefs_themes_btn_install_clicked_cb(GtkWidget *widget, gpointer data
        }
        g_free(alert_title);
        if (cinfo->dest == NULL) {
-               cinfo->dest = g_strconcat(get_home_dir(), G_DIR_SEPARATOR_S, RC_DIR,
-                                         G_DIR_SEPARATOR_S, PIXMAP_THEME_DIR, G_DIR_SEPARATOR_S,
+               cinfo->dest = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
+                                         PIXMAP_THEME_DIR, G_DIR_SEPARATOR_S,
                                          themename, NULL);
        }
        if (TRUE == is_dir_exist(cinfo->dest)) {
-               alertpanel_error(_("A theme with the same name is\nalready installed in this location"));
-               goto end_inst;
+               AlertValue val = alertpanel_full(_("Theme exists"),
+                               _("A theme with the same name is\nalready installed in this location.\n\n"
+                                 "Do you want to replace it?"),
+                               GTK_STOCK_CANCEL, _("Overwrite"), NULL, FALSE,
+                               NULL, ALERT_WARNING, G_ALERTDEFAULT);
+               if (val == G_ALERTALTERNATE) {
+                       if (remove_dir_recursive(cinfo->dest) < 0) {
+                               alertpanel_error(_("Couldn't delete the old theme in %s."), cinfo->dest);
+                               goto end_inst;
+                       }
+               } else {
+                       goto end_inst;
+               }
        }
        if (0 != make_dir_hier(cinfo->dest)) {
-               alertpanel_error(_("Couldn't create destination directory"));
+               alertpanel_error(_("Couldn't create destination directory %s."), cinfo->dest);
                goto end_inst;
        }
        prefs_themes_foreach_file(source, prefs_themes_file_install, cinfo);
@@ -559,9 +583,9 @@ static void prefs_themes_btn_install_clicked_cb(GtkWidget *widget, gpointer data
                                            (gpointer)(cinfo->dest), 
                                            (GCompareFunc)strcmp2);
                if (NULL != insted) {
-                       alertpanel_notice(_("Theme installed succesfully"));
+                       alertpanel_notice(_("Theme installed successfully."));
                        tdata->displayed = (gchar *)(insted->data);
-                       prefs_themes_set_themes_menu(GTK_OPTION_MENU(tdata->page->op_menu), tdata);
+                       prefs_themes_set_themes_menu(GTK_COMBO_BOX(tdata->page->op_menu), tdata);
                        prefs_themes_display_global_stats(tdata);
                        prefs_themes_get_theme_info(tdata);
                }
@@ -571,7 +595,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);
@@ -582,14 +606,91 @@ end_inst:
 static void prefs_themes_menu_item_activated_cb(GtkWidget *widget, gpointer data)
 {
        ThemesData *tdata = prefs_themes_data;
-       gchar      *path = (gchar *)data;
-       
-       g_return_if_fail(path != NULL);
-       
+       gchar      *path;
+       GtkTreeModel *model;
+       GtkTreeIter iter;
+
+       cm_return_if_fail(gtk_combo_box_get_active_iter(GTK_COMBO_BOX(widget), &iter));
+
+       model = gtk_combo_box_get_model(GTK_COMBO_BOX(widget));
+       gtk_tree_model_get(model, &iter, 1, &path, -1);
+
        tdata->displayed = path;
        prefs_themes_get_theme_info(tdata);
 }
 
+#ifdef HAVE_SVG
+#define MM_INCH 0.0393700787402
+static gdouble prefs_themes_compute_ppi(GdkScreen *screen)
+{
+       gdouble wp = gdk_screen_get_width(screen);
+       gdouble hp = gdk_screen_get_height(screen);
+       gdouble wi = gdk_screen_get_width_mm(screen);
+       gdouble hi = gdk_screen_get_height_mm(screen);
+       gdouble dp, di;
+
+       debug_print("screen is %f x %f pixels, %f x %f mm\n", wp, hp, wi, hi);
+
+       /* https://en.wikipedia.org/wiki/Pixel_density */
+       wi *= MM_INCH;
+       hi *= MM_INCH;
+       dp = sqrt(wp * wp + hp * hp);
+       di = sqrt(wi * wi + hi * hi);
+
+       return (di != 0.0)? dp / di: 0.0;
+}
+
+static gdouble prefs_themes_get_adjusted_ppi(void)
+{
+       gdouble ppi, cppi;
+       GdkScreen * screen = gdk_screen_get_default();
+
+       if (screen == NULL) { /* oops! */
+               g_warning("unable to get default GDK screen");
+               return MIN_PPI;
+       }
+
+       ppi = gdk_screen_get_resolution(screen);
+       cppi = prefs_themes_compute_ppi(screen);
+       debug_print("returned PPI: %f / computed PPI: %f\n", ppi, cppi);
+       /*
+        gdk_screen_get_resolution doesn't seem to work well when running
+        on a remote display and returns the value of the local display.
+        height/width functions do this better, so we can compute a PPI
+        from them and take the highest value.
+       */
+       return MAX(ppi, cppi);
+}
+
+static void prefs_themes_checkbtn_enable_scaling_toggled_cb (GtkWidget *widget, gpointer data)
+{
+       ThemesPage *page = (ThemesPage *) data;
+       gboolean enabled = gtk_toggle_button_get_active(
+                               GTK_TOGGLE_BUTTON (widget));
+       gboolean automatic = gtk_toggle_button_get_active(
+                               GTK_TOGGLE_BUTTON (page->checkbtn_scaling_auto));
+
+       gtk_widget_set_sensitive(page->checkbtn_scaling_auto, enabled);
+       gtk_widget_set_sensitive(page->spinbtn_scaling_ppi, enabled && !automatic);
+       gtk_widget_set_sensitive(page->label_scaling_ppi, enabled && !automatic);
+}
+
+static void prefs_themes_checkbtn_scaling_auto_toggled_cb(GtkWidget *widget, gpointer data)
+{
+       ThemesPage *page = (ThemesPage *) data;
+       gboolean automatic = gtk_toggle_button_get_active(
+                               GTK_TOGGLE_BUTTON (widget));
+
+       gtk_widget_set_sensitive(page->spinbtn_scaling_ppi, !automatic);
+       gtk_widget_set_sensitive(page->label_scaling_ppi, !automatic);
+
+       if (automatic) /* update PPI value */
+               gtk_spin_button_set_value(
+                               GTK_SPIN_BUTTON (page->spinbtn_scaling_ppi),
+                               prefs_themes_get_adjusted_ppi());
+}
+#endif
+
 static void prefs_themes_update_buttons(const ThemesData *tdata)
 {
        ThemesPage *theme = tdata->page;
@@ -597,9 +698,7 @@ static void prefs_themes_update_buttons(const ThemesData *tdata)
 
        can_use = !IS_CURRENT_THEME(tdata->displayed);
        can_rem = can_use && !IS_INTERNAL_THEME(tdata->displayed);
-       
-       if (theme->btn_use != NULL)
-               gtk_widget_set_sensitive(theme->btn_use, can_use);
+
        if (theme->btn_remove != NULL)
                gtk_widget_set_sensitive(theme->btn_remove, can_rem);
 }
@@ -633,10 +732,9 @@ static void prefs_themes_display_theme_info(ThemesData *tdata, const ThemeInfo *
        save_prefs_path = prefs_common.pixmap_theme_path;
        prefs_common.pixmap_theme_path = tdata->displayed;
        for (i = 0; i < PREVIEW_ICONS; ++i) {
-               stock_pixmap_gdk(theme->window, prefs_themes_icons[i], 
-                               &(theme->pixmaps[i]), &(theme->masks[i]));
-               gtk_image_set_from_pixmap(GTK_IMAGE(theme->icons[i]),
-                               theme->pixmaps[i], theme->masks[i]);
+               stock_pixbuf_gdk(prefs_themes_icons[i], &(theme->pixbufs[i]));
+               gtk_image_set_from_pixbuf(GTK_IMAGE(theme->icons[i]),
+                               theme->pixbufs[i]);
        }
        prefs_common.pixmap_theme_path = save_prefs_path;
 
@@ -677,12 +775,11 @@ static void prefs_themes_display_global_stats(const ThemesData *tdata)
 
 #define FGETS_INFOFILE_LINE() \
        line[0] = '\0'; \
-       fgets(line, INFOFILE_LINE_LEN, finfo); \
-       if ((len = strlen(line)) > 0) { \
+       if (fgets(line, INFOFILE_LINE_LEN, finfo) != NULL && (len = strlen(line)) > 0) { \
                if (line[len - 1] == '\n') line[len - 1] = '\0'; \
        } \
        else { \
-               strcpy(line, _("Unknown")); \
+               g_strlcpy(line, _("Unknown"),sizeof(line)); \
        }
 
 static void prefs_themes_get_theme_info(ThemesData *tdata)
@@ -695,17 +792,17 @@ static void prefs_themes_get_theme_info(ThemesData *tdata)
        ThemeInfo *info;
        ThemesPage *theme = tdata->page;
 
-       g_return_if_fail(theme != NULL);
+       cm_return_if_fail(theme != NULL);
        path = tdata->displayed;
-       g_return_if_fail(path != NULL);
+       cm_return_if_fail(path != NULL);
 
        debug_print("Getting theme info for %s\n", path);
-       
+
        info = g_new0(ThemeInfo, 1);
-       
+
        if (IS_INTERNAL_THEME(path)) {
                info->name = g_strdup(_("Default internal theme"));
-               info->author = g_strdup(_("The Sylpheed-Claws Team"));
+               info->author = g_strdup(_("The Claws Mail Team"));
                info->url = g_strdup(HOMEPAGE_URI);
                info->status = g_strdup_printf(_("Internal theme has %d icons"), N_STOCK_PIXMAPS);
        }
@@ -731,7 +828,7 @@ static void prefs_themes_get_theme_info(ThemesData *tdata)
 
                info->status = prefs_themes_get_theme_stats(path);
                if (info->status == NULL) {
-                       info->status = g_strdup(_("Error: can't get theme status"));
+                       info->status = g_strdup(_("Error: couldn't get theme status"));
                }
        }
 
@@ -741,7 +838,7 @@ static void prefs_themes_get_theme_info(ThemesData *tdata)
        g_free(info->author);
        g_free(info->url);
        g_free(info->status);
-       
+
        g_free(info);
 }
 
@@ -751,51 +848,24 @@ static gchar *prefs_themes_get_theme_stats(const gchar *dirname)
 {
        gchar   *stats;
        DirInfo *dinfo;
+       gint     i;
 
        dinfo = g_new0(DirInfo, 1);
-       
+       dinfo->supported = stock_pixmap_theme_extensions();
+       for (i = 0; (dinfo->supported)[i] != NULL; ++i);
+       dinfo->length = g_malloc(i * sizeof(gint));
+       for (i = 0; (dinfo->supported)[i] != NULL; ++i) {
+               (dinfo->length)[i] = strlen((dinfo->supported)[i]);
+       }
        prefs_themes_foreach_file(dirname, prefs_themes_file_stats, dinfo);
-       stats = g_strdup_printf(_("%d files (%d icons), size: %s"), 
-                               dinfo->files, dinfo->pixms, to_human_readable(dinfo->bytes));
-       
+       stats = g_strdup_printf(_("%d files (%d icons), size: %s"),
+                               dinfo->files, dinfo->pixms,
+                               to_human_readable((goffset)dinfo->bytes));
+       g_free(dinfo->length);
        g_free(dinfo);
        return stats;
 }
 
-/* BEGIN GLADE CODE */
-/* This is a dummy pixmap we use when a pixmap can't be found. */
-static char *dummy_pixmap_xpm[] = {
-       /* columns rows colors chars-per-pixel */
-       "1 1 1 1",
-       "  c None",
-       /* pixels */
-       " "
-};
-
-/* This is an internally used function to create pixmaps. */
-static GtkWidget* create_dummy_pixmap(GtkWidget *widget)
-{
-       GdkColormap *colormap;
-       GdkPixmap *gdkpixmap;
-       GdkBitmap *mask;
-       GtkWidget *pixmap;
-
-       colormap = gtk_widget_get_colormap (widget);
-       gdkpixmap = gdk_pixmap_colormap_create_from_xpm_d (NULL, colormap, &mask,
-                       NULL, dummy_pixmap_xpm);
-       if (gdkpixmap == NULL)
-               g_error ("Couldn't create replacement pixmap.");
-       pixmap = gtk_image_new_from_pixmap(gdkpixmap, mask);
-       g_object_unref (gdkpixmap);
-       g_object_unref (mask);
-       return pixmap;
-}
-/* END GLADE CODE */
-
-/* glade generates some calls to a create_pixmap support function 
- * we don't really need. */
-#define create_pixmap(widget,filename) create_dummy_pixmap(widget)
-
 static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpointer data)
 {
        ThemesPage *prefs_themes = (ThemesPage *)page;
@@ -806,8 +876,6 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
        GtkWidget *vbox2;
        GtkWidget *hbox3;
        GtkWidget *menu_themes;
-       GtkWidget *menu_themes_menu;
-       GtkWidget *glade_menuitem;
        GtkWidget *btn_install;
        GtkWidget *btn_more;
        GtkWidget *label_global_status;
@@ -832,40 +900,39 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
        GtkWidget *icon_7;
        GtkWidget *frame_buttons;
        GtkWidget *hbuttonbox1;
-       GtkWidget *btn_use;
        GtkWidget *btn_remove;
+       GtkCellRenderer *renderer;
+#ifdef HAVE_SVG
+       GtkWidget *frame_scaling;
+       GtkWidget *checkbtn_enable_alpha;
+       GtkWidget *checkbtn_enable_scaling;
+       GtkWidget *checkbtn_scaling_auto;
+       GtkWidget *label_scaling_ppi;
+       GtkWidget *spinbtn_scaling_ppi;
+       GtkAdjustment *spinbtn_scaling_ppi_adj;
+#endif
 
        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, VSPACING);
-       gtk_container_set_border_width (GTK_CONTAINER (vbox2), VBOX_BORDER);
-       gtk_widget_show (vbox2);
-       gtk_container_add (GTK_CONTAINER (frame1), vbox2);
+       vbox2 = gtkut_get_options_frame(vbox1, &frame1, _("Selector"));
 
        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);
 
-       menu_themes = gtk_option_menu_new ();
+       menu_themes = gtk_combo_box_new();
        gtk_widget_show (menu_themes);
        gtk_box_pack_start (GTK_BOX (hbox3), menu_themes, FALSE, FALSE, 0);
-       menu_themes_menu = gtk_menu_new ();
-       glade_menuitem = gtk_menu_item_new_with_label ("");
-       gtk_widget_show (glade_menuitem);
-       gtk_menu_append (GTK_MENU (menu_themes_menu), glade_menuitem);
-       gtk_option_menu_set_menu (GTK_OPTION_MENU (menu_themes), menu_themes_menu);
 
        btn_install = gtk_button_new_with_label (_("Install new..."));
        gtk_widget_show (btn_install);
        gtk_box_pack_start (GTK_BOX (hbox3), btn_install, FALSE, FALSE, 0);
-       GTK_WIDGET_SET_FLAGS (btn_install, GTK_CAN_DEFAULT);
+       gtkut_widget_set_can_default (btn_install, TRUE);
 
-       btn_more = gtkut_get_link_btn(window, THEMES_URI, _("Get more..."));
+       btn_more = gtkut_get_link_btn((GtkWidget *)window, THEMES_URI, _("Get more..."));
        gtk_widget_show (btn_more);
        gtk_box_pack_start (GTK_BOX (hbox3), btn_more, FALSE, FALSE, 0);
 
@@ -946,37 +1013,37 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
        gtk_widget_show (hbox1);
        gtk_container_add (GTK_CONTAINER (frame_preview), hbox1);
 
-       icon_1 = create_pixmap (vbox1, NULL);
+       icon_1 = gtk_image_new();
        gtk_widget_show (icon_1);
        gtk_box_pack_start (GTK_BOX (hbox1), icon_1, TRUE, TRUE, 2);
        gtk_misc_set_padding (GTK_MISC (icon_1), 0, 5);
 
-       icon_2 = create_pixmap (vbox1, NULL);
+       icon_2 = gtk_image_new();
        gtk_widget_show (icon_2);
        gtk_box_pack_start (GTK_BOX (hbox1), icon_2, TRUE, TRUE, 2);
        gtk_misc_set_padding (GTK_MISC (icon_2), 0, 5);
 
-       icon_3 = create_pixmap (vbox1, NULL);
+       icon_3 = gtk_image_new();
        gtk_widget_show (icon_3);
        gtk_box_pack_start (GTK_BOX (hbox1), icon_3, TRUE, TRUE, 2);
        gtk_misc_set_padding (GTK_MISC (icon_3), 0, 5);
 
-       icon_4 = create_pixmap (vbox1, NULL);
+       icon_4 = gtk_image_new();
        gtk_widget_show (icon_4);
        gtk_box_pack_start (GTK_BOX (hbox1), icon_4, TRUE, TRUE, 2);
        gtk_misc_set_padding (GTK_MISC (icon_4), 0, 5);
 
-       icon_5 = create_pixmap (vbox1, NULL);
+       icon_5 = gtk_image_new();
        gtk_widget_show (icon_5);
        gtk_box_pack_start (GTK_BOX (hbox1), icon_5, TRUE, TRUE, 2);
        gtk_misc_set_padding (GTK_MISC (icon_5), 0, 5);
 
-       icon_6 = create_pixmap (vbox1, NULL);
+       icon_6 = gtk_image_new();
        gtk_widget_show (icon_6);
        gtk_box_pack_start (GTK_BOX (hbox1), icon_6, TRUE, TRUE, 0);
        gtk_misc_set_padding (GTK_MISC (icon_6), 0, 5);
 
-       icon_7 = create_pixmap (vbox1, NULL);
+       icon_7 = gtk_image_new();
        gtk_widget_show (icon_7);
        gtk_box_pack_start (GTK_BOX (hbox1), icon_7, TRUE, TRUE, 0);
        gtk_misc_set_padding (GTK_MISC (icon_7), 0, 5);
@@ -988,22 +1055,72 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
        gtk_container_add (GTK_CONTAINER (frame_buttons), hbuttonbox1);
        gtk_container_set_border_width (GTK_CONTAINER (hbuttonbox1), 5);
        gtk_button_box_set_layout (GTK_BUTTON_BOX (hbuttonbox1), GTK_BUTTONBOX_START);
-       gtk_button_box_set_spacing (GTK_BUTTON_BOX (hbuttonbox1), 5);
-       gtk_button_box_set_child_ipadding (GTK_BUTTON_BOX (hbuttonbox1), 5, 0);
-
-       btn_use = gtk_button_new_with_label (_("Use this"));
-       gtk_widget_show (btn_use);
-       gtk_container_add (GTK_CONTAINER (hbuttonbox1), btn_use);
-       GTK_WIDGET_SET_FLAGS (btn_use, GTK_CAN_DEFAULT);
+       gtk_box_set_spacing (GTK_BOX (hbuttonbox1), 5);
 
        btn_remove = gtk_button_new_with_label (_("Remove"));
        gtk_widget_show (btn_remove);
        gtk_container_add (GTK_CONTAINER (hbuttonbox1), btn_remove);
-       GTK_WIDGET_SET_FLAGS (btn_remove, GTK_CAN_DEFAULT);
+       gtkut_widget_set_can_default (btn_remove, TRUE);
+
+#ifdef HAVE_SVG
+       PACK_FRAME(vbox1, frame_scaling, _("SVG rendering"));
+
+       vbox2 = gtk_vbox_new (FALSE, VSPACING);
+       gtk_widget_show (vbox2);
+       gtk_container_set_border_width (GTK_CONTAINER (vbox2), 5);
+       gtk_container_add (GTK_CONTAINER (frame_scaling), vbox2);
+
+       PACK_CHECK_BUTTON(vbox2, checkbtn_enable_alpha, _("Enable alpha channel"));
+       PACK_CHECK_BUTTON(vbox2, checkbtn_enable_scaling, _("Force scaling"));
+       PACK_CHECK_BUTTON(vbox2, checkbtn_scaling_auto, _("Automatic"));
+
+       hbox3 = gtk_hbox_new (FALSE, 5);
+       gtk_widget_show (hbox3);
+
+       label_scaling_ppi = gtk_label_new (_("Pixels per inch (PPI)"));
+       gtk_widget_show (label_scaling_ppi);
+       gtk_box_pack_start (GTK_BOX (hbox3), label_scaling_ppi,
+                       FALSE, FALSE, 5);
+
+       spinbtn_scaling_ppi_adj = GTK_ADJUSTMENT(
+               gtk_adjustment_new (MIN_PPI, MIN_PPI, MAX_PPI, 1, 10, 0));
+       spinbtn_scaling_ppi = gtk_spin_button_new(
+                       spinbtn_scaling_ppi_adj, 1.0, 0);
+       gtk_widget_show (spinbtn_scaling_ppi);
+       gtk_box_pack_start (GTK_BOX (hbox3), spinbtn_scaling_ppi,
+                       FALSE, FALSE, 5);
+
+       gtk_box_pack_start (GTK_BOX (vbox2), hbox3, FALSE, FALSE, 0);
+
+       /* initialize widget values */
+       gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON (checkbtn_enable_alpha),
+                       prefs_common.enable_alpha_svg);
+       gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON (checkbtn_enable_scaling),
+                       prefs_common.enable_pixmap_scaling);
+       gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON (checkbtn_scaling_auto),
+                       prefs_common.pixmap_scaling_auto);
+       gtk_spin_button_set_value(GTK_SPIN_BUTTON (spinbtn_scaling_ppi),
+                       prefs_common.pixmap_scaling_ppi);
+
+       /* sensitivity */
+       gtk_widget_set_sensitive(checkbtn_scaling_auto,
+                       prefs_common.enable_pixmap_scaling);
+       gtk_widget_set_sensitive(spinbtn_scaling_ppi,
+                       prefs_common.enable_pixmap_scaling
+                               && !prefs_common.pixmap_scaling_auto);
+       gtk_widget_set_sensitive(label_scaling_ppi,
+                       prefs_common.enable_pixmap_scaling
+                               && !prefs_common.pixmap_scaling_auto);
+
+       /* signals */
+       g_signal_connect(G_OBJECT(checkbtn_enable_scaling), "toggled",
+                        G_CALLBACK(prefs_themes_checkbtn_enable_scaling_toggled_cb),
+                        prefs_themes);
+       g_signal_connect(G_OBJECT(checkbtn_scaling_auto), "toggled",
+                        G_CALLBACK(prefs_themes_checkbtn_scaling_auto_toggled_cb),
+                        prefs_themes);
+#endif
 
-       g_signal_connect(G_OBJECT(btn_use), "clicked",
-                        G_CALLBACK(prefs_themes_btn_use_clicked_cb),
-                        NULL);
        g_signal_connect(G_OBJECT(btn_remove), "clicked",
                         G_CALLBACK(prefs_themes_btn_remove_clicked_cb),
                         NULL);
@@ -1012,7 +1129,7 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
                         NULL);
 
        prefs_themes->window = GTK_WIDGET(window);
-       
+
        prefs_themes->name   = label_name;
        prefs_themes->author = label_author;
        prefs_themes->url    = label_url;
@@ -1026,18 +1143,29 @@ static void prefs_themes_create_widget(PrefsPage *page, GtkWindow *window, gpoin
        prefs_themes->icons[4] = icon_5;
        prefs_themes->icons[5] = icon_6;
        prefs_themes->icons[6] = icon_7;
-       
-       prefs_themes->btn_use     = btn_use;
+
        prefs_themes->btn_remove  = btn_remove;
        prefs_themes->btn_install = btn_install;
        prefs_themes->btn_more    = btn_more;
 
        prefs_themes->op_menu     = menu_themes;
 
-       prefs_themes->page.widget = vbox1;
+#ifdef HAVE_SVG
+       prefs_themes->checkbtn_enable_alpha   = checkbtn_enable_alpha;
+       prefs_themes->checkbtn_enable_scaling = checkbtn_enable_scaling;
+       prefs_themes->checkbtn_scaling_auto   = checkbtn_scaling_auto;
+       prefs_themes->label_scaling_ppi       = label_scaling_ppi;
+       prefs_themes->spinbtn_scaling_ppi     = spinbtn_scaling_ppi;
+#endif
 
-       prefs_themes_set_themes_menu(GTK_OPTION_MENU(prefs_themes->op_menu), tdata);
+       prefs_themes->page.widget = vbox1;
        
+       prefs_themes_set_themes_menu(GTK_COMBO_BOX(menu_themes), tdata);
+       renderer = gtk_cell_renderer_text_new();
+       gtk_cell_layout_pack_start(GTK_CELL_LAYOUT(menu_themes), renderer, TRUE);
+       gtk_cell_layout_set_attributes(GTK_CELL_LAYOUT(menu_themes), renderer,
+                                       "text", 0, NULL);
+
        prefs_themes_get_theme_info(tdata);
        prefs_themes_display_global_stats(tdata);
 }
@@ -1049,6 +1177,49 @@ static void prefs_themes_destroy_widget(PrefsPage *page)
 
 static void prefs_themes_save(PrefsPage *page)
 {
-       /* ThemesPage *theme = (ThemesPage *)page; */
+       ThemesData *tdata = prefs_themes_data;
+       gchar      *theme_str = tdata->displayed;
+#ifdef HAVE_SVG
+       ThemesPage *tpage = (ThemesPage *) page;
+       gboolean alpha = prefs_common.enable_alpha_svg;
+       gboolean scaling = prefs_common.enable_pixmap_scaling;
+       gboolean scaling_auto = prefs_common.pixmap_scaling_auto;
+       gint scaling_ppi = prefs_common.pixmap_scaling_ppi;
+
+       prefs_common.enable_alpha_svg = gtk_toggle_button_get_active(
+               GTK_TOGGLE_BUTTON (tpage->checkbtn_enable_alpha));
+       prefs_common.enable_pixmap_scaling = gtk_toggle_button_get_active(
+               GTK_TOGGLE_BUTTON (tpage->checkbtn_enable_scaling));
+       prefs_common.pixmap_scaling_auto = gtk_toggle_button_get_active(
+               GTK_TOGGLE_BUTTON (tpage->checkbtn_scaling_auto));
+       prefs_common.pixmap_scaling_ppi = gtk_spin_button_get_value_as_int (
+               GTK_SPIN_BUTTON (tpage->spinbtn_scaling_ppi));
+#endif
+
+       if (!IS_CURRENT_THEME(theme_str)) {
+               debug_print("Changing theme to %s\n", theme_str);
+               g_free(prefs_common.pixmap_theme_path);
+
+               prefs_common.pixmap_theme_path = g_strdup(theme_str);
+
+               main_window_reflect_prefs_all_real(TRUE);
+               compose_reflect_prefs_pixmap_theme();
+               addrcompl_reflect_prefs_pixmap_theme();
+
+               prefs_themes_update_buttons(tdata);
+       }
+#ifdef HAVE_SVG
+       else if (scaling != prefs_common.enable_pixmap_scaling
+                       || alpha != prefs_common.enable_alpha_svg
+                       || (scaling_auto != prefs_common.pixmap_scaling_auto
+                               && scaling_ppi != prefs_common.pixmap_scaling_ppi)) {
+               /* same theme, different scaling options */
+               debug_print("Updating theme scaling\n");
+               stock_pixmap_invalidate_all_icons();
+               main_window_reflect_prefs_all_real(TRUE);
+               compose_reflect_prefs_pixmap_theme();
+               addrcompl_reflect_prefs_pixmap_theme();
+       }
+#endif
 }