[gtk+/wip/cosimoc/menuitem-gadget: 8/8] checkmenuitem: deprecate indicator-size style property



commit 6783a443a5da90961a4de66621f943bc5ebce99b
Author: Cosimo Cecchi <cosimoc gnome org>
Date:   Fri Dec 18 17:59:34 2015 -0800

    checkmenuitem: deprecate indicator-size style property
    
    We can now use min-width/min-height.

 gtk/gtkcheckmenuitem.c                   |   32 +++++++++--------------------
 gtk/theme/Adwaita/_common.scss           |    3 +-
 gtk/theme/Adwaita/gtk-contained-dark.css |   12 ++++++----
 gtk/theme/Adwaita/gtk-contained.css      |   12 ++++++----
 4 files changed, 26 insertions(+), 33 deletions(-)
---
diff --git a/gtk/gtkcheckmenuitem.c b/gtk/gtkcheckmenuitem.c
index 2435016..d83c7b4 100644
--- a/gtk/gtkcheckmenuitem.c
+++ b/gtk/gtkcheckmenuitem.c
@@ -158,26 +158,6 @@ gtk_check_menu_item_render_indicator (GtkCssGadget *gadget,
 }
 
 static void
-gtk_check_menu_item_measure_indicator (GtkCssGadget   *gadget,
-                                       GtkOrientation  orientation,
-                                       int             size,
-                                       int            *minimum,
-                                       int            *natural,
-                                       int            *minimum_baseline,
-                                       int            *natural_baseline,
-                                       gpointer        data)
-{
-  GtkWidget *widget = gtk_css_gadget_get_owner (gadget);
-  guint indicator_size;
-
-  gtk_widget_style_get (widget,
-                        "indicator-size", &indicator_size,
-                        NULL);
-
-  *minimum = *natural = indicator_size;
-}
-
-static void
 gtk_check_menu_item_size_allocate (GtkWidget     *widget,
                                    GtkAllocation *allocation)
 {
@@ -279,7 +259,15 @@ gtk_check_menu_item_class_init (GtkCheckMenuItemClass *klass)
                                                          P_("Whether the menu item looks like a radio menu 
item"),
                                                          FALSE,
                                                          GTK_PARAM_READWRITE|G_PARAM_EXPLICIT_NOTIFY));
-  
+
+  /**
+   * GtkCheckMenuItem:indicator-size:
+   *
+   * The size of the check or radio indicator.
+   *
+   * Deprecated: 3.20: Use the standard CSS property min-width on the check or
+   *   radio nodes; the value of this style property is ignored.
+   */
   gtk_widget_class_install_style_property (widget_class,
                                            g_param_spec_int ("indicator-size",
                                                              P_("Indicator Size"),
@@ -670,7 +658,7 @@ gtk_check_menu_item_init (GtkCheckMenuItem *check_menu_item)
   priv->indicator_gadget =
     gtk_css_custom_gadget_new_for_node (priv->indicator_node,
                                         GTK_WIDGET (check_menu_item),
-                                        gtk_check_menu_item_measure_indicator,
+                                        NULL,
                                         NULL,
                                         gtk_check_menu_item_render_indicator,
                                         NULL, NULL);
diff --git a/gtk/theme/Adwaita/_common.scss b/gtk/theme/Adwaita/_common.scss
index 89342b0..2578279 100644
--- a/gtk/theme/Adwaita/_common.scss
+++ b/gtk/theme/Adwaita/_common.scss
@@ -11,7 +11,6 @@ $asset_suffix: if($variant=='dark', '-dark', '');
   -GtkTextView-error-underline-color: $error_color;
 
   -GtkCheckButton-indicator-size: 16;
-  -GtkCheckMenuItem-indicator-size: 16;
 
   // The size for scrollbars. The slider is 2px smaller, but we keep it
   // up so that the whole area is sensitive to button presses for the
@@ -1544,6 +1543,8 @@ menuitem {
   & accelerator { color: gtkalpha(currentColor,0.55); }
   & check,
   & radio {
+    min-height: 16px;
+    min-width: 16px;
     &:dir(ltr) { margin-right: 7px; }
     &:dir(rtl) { margin-left: 7px; }
   }
diff --git a/gtk/theme/Adwaita/gtk-contained-dark.css b/gtk/theme/Adwaita/gtk-contained-dark.css
index fe8a9e1..757f4cd 100644
--- a/gtk/theme/Adwaita/gtk-contained-dark.css
+++ b/gtk/theme/Adwaita/gtk-contained-dark.css
@@ -3,7 +3,6 @@
   -GtkToolButton-icon-spacing: 4;
   -GtkTextView-error-underline-color: #cc0000;
   -GtkCheckButton-indicator-size: 16;
-  -GtkCheckMenuItem-indicator-size: 16;
   -GtkScrolledWindow-scrollbar-spacing: 0;
   -GtkScrolledWindow-scrollbars-within-bevel: 1;
   -GtkToolItemGroup-expander-size: 11;
@@ -2148,10 +2147,13 @@ menu,
 
 menuitem accelerator {
   color: alpha(currentColor,0.55); }
-menuitem check:dir(ltr), menuitem radio:dir(ltr) {
-  margin-right: 7px; }
-menuitem check:dir(rtl), menuitem radio:dir(rtl) {
-  margin-left: 7px; }
+menuitem check, menuitem radio {
+  min-height: 16px;
+  min-width: 16px; }
+  menuitem check:dir(ltr), menuitem radio:dir(ltr) {
+    margin-right: 7px; }
+  menuitem check:dir(rtl), menuitem radio:dir(rtl) {
+    margin-left: 7px; }
 
 /***************
  * Popovers   *
diff --git a/gtk/theme/Adwaita/gtk-contained.css b/gtk/theme/Adwaita/gtk-contained.css
index 7ba0b2e..60c9792 100644
--- a/gtk/theme/Adwaita/gtk-contained.css
+++ b/gtk/theme/Adwaita/gtk-contained.css
@@ -3,7 +3,6 @@
   -GtkToolButton-icon-spacing: 4;
   -GtkTextView-error-underline-color: #cc0000;
   -GtkCheckButton-indicator-size: 16;
-  -GtkCheckMenuItem-indicator-size: 16;
   -GtkScrolledWindow-scrollbar-spacing: 0;
   -GtkScrolledWindow-scrollbars-within-bevel: 1;
   -GtkToolItemGroup-expander-size: 11;
@@ -2154,10 +2153,13 @@ menu,
 
 menuitem accelerator {
   color: alpha(currentColor,0.55); }
-menuitem check:dir(ltr), menuitem radio:dir(ltr) {
-  margin-right: 7px; }
-menuitem check:dir(rtl), menuitem radio:dir(rtl) {
-  margin-left: 7px; }
+menuitem check, menuitem radio {
+  min-height: 16px;
+  min-width: 16px; }
+  menuitem check:dir(ltr), menuitem radio:dir(ltr) {
+    margin-right: 7px; }
+  menuitem check:dir(rtl), menuitem radio:dir(rtl) {
+    margin-left: 7px; }
 
 /***************
  * Popovers   *


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]