From b09225b1a1004bf9562e88d329bb82619894652b Mon Sep 17 00:00:00 2001
From: Sven Neumann <sven@gimp.org>
Date: Tue, 12 Oct 2004 10:37:11 +0000
Subject: [PATCH] use a larger icon size for GimpImagefile views.

2004-10-12  Sven Neumann  <sven@gimp.org>

	* app/widgets/gimpaction.c (gimp_action_set_proxy): use a larger
	icon size for GimpImagefile views.

	* themes/Default/images/stock-frame-64.png: removed the 1 pixel
	wide empty border around the frame.

	* app/widgets/gimpviewrenderer-frame.c: adjusted the hardcoded values.
---
 ChangeLog                                |  10 +++++++
 app/widgets/gimpaction.c                 |  33 +++++++++++++----------
 app/widgets/gimpviewrenderer-frame.c     |   8 +++---
 themes/Default/images/stock-frame-64.png | Bin 405 -> 398 bytes
 4 files changed, 33 insertions(+), 18 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 41753e361f..ef3b3419fb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2004-10-12  Sven Neumann  <sven@gimp.org>
+
+	* app/widgets/gimpaction.c (gimp_action_set_proxy): use a larger
+	icon size for GimpImagefile views.
+
+	* themes/Default/images/stock-frame-64.png: removed the 1 pixel
+	wide empty border around the frame.
+
+	* app/widgets/gimpviewrenderer-frame.c: adjusted the hardcoded values.
+
 2004-10-12  Sven Neumann  <sven@gimp.org>
 
 	* Makefile.am: defined DISTCHECK_CONFIGURE_FLAGS with the
diff --git a/app/widgets/gimpaction.c b/app/widgets/gimpaction.c
index 78ba31e2c5..62a2f126e3 100644
--- a/app/widgets/gimpaction.c
+++ b/app/widgets/gimpaction.c
@@ -275,16 +275,14 @@ gimp_action_set_proxy (GimpAction *action,
 
       if (! area)
         {
-          GdkScreen   *screen;
-          GtkSettings *settings;
+          GdkScreen   *screen   = gtk_widget_get_screen (proxy);
+          GtkSettings *settings = gtk_settings_get_for_screen (screen);
           gint         width, height;
 
           area = gimp_color_area_new (action->color,
                                       GIMP_COLOR_AREA_SMALL_CHECKS, 0);
           gimp_color_area_set_draw_border (GIMP_COLOR_AREA (area), TRUE);
 
-          screen = gtk_widget_get_screen (area);
-          settings = gtk_settings_get_for_screen (screen);
           gtk_icon_size_lookup_for_settings (settings, GTK_ICON_SIZE_MENU,
                                              &width, &height);
 
@@ -311,20 +309,27 @@ gimp_action_set_proxy (GimpAction *action,
 
       if (! view)
         {
-          GdkScreen *screen;
-          gint       width, height;
-          gint       border_width;
+          GdkScreen   *screen   = gtk_widget_get_screen (proxy);
+          GtkSettings *settings = gtk_settings_get_for_screen (screen);
+          GtkIconSize  size;
+          gint         width, height;
+          gint         border_width;
 
-          screen = gtk_widget_get_screen (proxy);
-          gtk_icon_size_lookup_for_settings (gtk_settings_get_for_screen (screen),
-                                             GTK_ICON_SIZE_MENU,
-                                             &width, &height);
+          if (GIMP_IS_IMAGEFILE (action->viewable))
+            {
+              size         = GTK_ICON_SIZE_LARGE_TOOLBAR;
+              border_width = 0;
+            }
+          else
+            {
+              size         = GTK_ICON_SIZE_MENU;
+              border_width = 1;
+            }
 
-          /*  FIXME: remove this hack  */
-          border_width = GIMP_IS_IMAGEFILE (action->viewable) ? 0 : 1;
+          gtk_icon_size_lookup_for_settings (settings, size, &width, &height);
 
           view = gimp_view_new_full (action->viewable,
-                                     width - 2, height - 2, border_width,
+                                     width, height, border_width,
                                      FALSE, FALSE, FALSE);
           gtk_image_menu_item_set_image (GTK_IMAGE_MENU_ITEM (proxy), view);
           gtk_widget_show (view);
diff --git a/app/widgets/gimpviewrenderer-frame.c b/app/widgets/gimpviewrenderer-frame.c
index c666a51775..b5a644d785 100644
--- a/app/widgets/gimpviewrenderer-frame.c
+++ b/app/widgets/gimpviewrenderer-frame.c
@@ -205,10 +205,10 @@ gimp_view_renderer_ensure_frame (GimpViewRenderer *renderer,
                                              GTK_ICON_SIZE_DIALOG, NULL);
 
       /*  FIXME: shouldn't be hardcoded  */
-      class->frame_left   = 3;
-      class->frame_top    = 3;
-      class->frame_right  = 5;
-      class->frame_bottom = 5;
+      class->frame_left   = 2;
+      class->frame_top    = 2;
+      class->frame_right  = 4;
+      class->frame_bottom = 4;
     }
 }
 
diff --git a/themes/Default/images/stock-frame-64.png b/themes/Default/images/stock-frame-64.png
index 747a3dad1ac771acaf3fa6a61dc5f9d2663d37ba..71513d74c2697925fc3e17a7a27fcc19b5413e1e 100644
GIT binary patch
literal 398
zcmeAS@N?(olHy`uVBq!ia0vp^4j|0I1|(Ny7TyC=Y)RhkE)4%caKYZ?lYt_f1s;*b
z3=G`DAk4@xYmNj^kiEpy*OmPW7Y~;L=aNfa0SpX`(w;7kAr-gY-rnoi;vmxYu>414
zl4?kq)ZYK67n*I{a&TjzxgUqvnOV8IPxr(dJ7k<YJaeHHr-uXgmjf%~)W5}C|3B|-
zT5Mzf0)}k%Ez?gg<vHxo@?_4;WiKTo8rJi@SS-SC(5HOBd#|o<?(d9B7PSXguGq@!
zDi^SCIors$YyXn{a^C_O>;4z<9u$^TaAkL3U}O;>i|CtvmTv_^^p*AeIf0B^zd7Ds
zW5_;d>GSln4Rgu^o;97d4Idh7=X~MZkv@O5H+#-+rW&UF1KJ<%U2XTSeYv>lj^E^E
p&2P%HZqLu2%N-ze@c8q3h74A711bIWAAljp;OXk;vd$@?2>^^Mijx2U

literal 405
zcmeAS@N?(olHy`uVBq!ia0vp^4j|0I1|(Ny7TyC=>?NMQuIyL17`a&_&L~;FV_;zP
z^mK6yskrs__HMr;1_G@Q>o=LN@X)dn*z^B!POppS<{53h92XaM@Sg5!D|%bJ#KSM>
zY<hZ-ug4-4B_@sQ-;?#Mt)jivUE7xg{ouO6aC=YRDF&7^2iOf5H><4ueEO<I-{rjR
zi=Qo-BGLAc(SqUqf#+85|1JCWi1F7x_KGvxgI&(V+C7TCbyC*g$*jc;YowT7G`_y{
ze3$*My0<synEZU9_43RAlk4|KyFbnpRfu9gz`$nEz{r!pz$`&JGpnbD=^VqA2k%xh
ztT1foOl=5i;5slTXw|+?f9v=sG$cD}2q%Xh&^;aS(6->PQLLz_>~GEr=6Q|k4_GCx
zPZALnK5e0SK-<2!RJHKoee25O{X6$D8+ZnNK79G&uOENJ`B^hWx9Br(2Zl3)r>mdK
II;Vst0EDHV`2YX_