Improve size of HildonColorButton

* hildon/hildon-color-button.c (hildon_color_button_init): Use
HILDON_ICON_PIXEL_SIZE_FINGER for the color icon and 80x70 for the button
itself, to make it fit better with Fremantle style.

Fixes: NB#109878 (height and width of focus on color icon is not
consistent with other item in the application tool bar)

diff --git a/ChangeLog b/ChangeLog
index 9b48374..2f6f9da 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2009-06-04  Claudio Saavedra  <csaavedra@igalia.com>
+
+       * hildon/hildon-color-button.c (hildon_color_button_init): Use
+       HILDON_ICON_PIXEL_SIZE_FINGER for the color icon and 80x70 for the button
+       itself, to make it fit better with Fremantle style.
+
+       Fixes: NB#109878 (height and width of focus on color icon is not
+       consistent with other item in the application tool bar)
+
 2009-06-02  Alejandro Pinheiro  <apinheiro@igalia.com>
 
        * hildon/hildon-touch-selector.c:

diff --git a/hildon/hildon-color-button.c b/hildon/hildon-color-button.c
index 49c0dfe..a9430ba 100644 (file)
--- a/hildon/hildon-color-button.c
+++ b/hildon/hildon-color-button.c
@@ -58,13 +58,9 @@
 #include                                        "hildon-color-chooser-dialog.h"
 #include                                        "hildon-color-button-private.h"
 
-#define                                         COLOR_FILLED_HEIGHT 22
+#define                                         COLOR_BUTTON_WIDTH 80
 
-#define                                         COLOR_FILLED_WIDTH 22
-
-#define                                         COLOR_BUTTON_WIDTH 52
-
-#define                                         COLOR_BUTTON_HEIGHT 48
+#define                                         COLOR_BUTTON_HEIGHT 70
 
 #define                                         OUTER_BORDER_RED 0
 
@@ -352,7 +348,8 @@ hildon_color_button_init                        (HildonColorButton *cb)
             COLOR_BUTTON_HEIGHT);
 
     gtk_widget_set_size_request (GTK_WIDGET(drawing_area),
-            COLOR_FILLED_WIDTH, COLOR_FILLED_HEIGHT);
+                                 HILDON_ICON_PIXEL_SIZE_FINGER,
+                                 HILDON_ICON_PIXEL_SIZE_FINGER);
 
     /* Connect the callback function for exposure event */
     g_signal_connect (drawing_area, "expose-event",