Revert "app: reorder spacing after size in the new spacing override code/ui"
This reverts commit 492e1ab560
,
but keeps the code cleanups, and adds some more.
This commit is contained in:
@ -40,9 +40,9 @@
|
||||
|
||||
|
||||
#define DEFAULT_BRUSH_SIZE 20.0
|
||||
#define DEFAULT_BRUSH_SPACING 10.0
|
||||
#define DEFAULT_BRUSH_ASPECT_RATIO 0.0
|
||||
#define DEFAULT_BRUSH_ANGLE 0.0
|
||||
#define DEFAULT_BRUSH_SPACING 10.0
|
||||
|
||||
#define DEFAULT_APPLICATION_MODE GIMP_PAINT_CONSTANT
|
||||
#define DEFAULT_HARD FALSE
|
||||
@ -78,9 +78,9 @@ enum
|
||||
PROP_USE_APPLICATOR, /* temp debug */
|
||||
|
||||
PROP_BRUSH_SIZE,
|
||||
PROP_BRUSH_SPACING,
|
||||
PROP_BRUSH_ASPECT_RATIO,
|
||||
PROP_BRUSH_ANGLE,
|
||||
PROP_BRUSH_SPACING,
|
||||
|
||||
PROP_APPLICATION_MODE,
|
||||
PROP_HARD,
|
||||
@ -159,11 +159,6 @@ gimp_paint_options_class_init (GimpPaintOptionsClass *klass)
|
||||
1.0, 10000.0, DEFAULT_BRUSH_SIZE,
|
||||
GIMP_PARAM_STATIC_STRINGS);
|
||||
|
||||
GIMP_CONFIG_INSTALL_PROP_DOUBLE (object_class, PROP_BRUSH_SPACING,
|
||||
"brush-spacing", _("Brush Spacing"),
|
||||
1.0, 5000.0, DEFAULT_BRUSH_SPACING,
|
||||
GIMP_PARAM_STATIC_STRINGS);
|
||||
|
||||
GIMP_CONFIG_INSTALL_PROP_DOUBLE (object_class, PROP_BRUSH_ASPECT_RATIO,
|
||||
"brush-aspect-ratio", _("Brush Aspect Ratio"),
|
||||
-20.0, 20.0, DEFAULT_BRUSH_ASPECT_RATIO,
|
||||
@ -174,6 +169,11 @@ gimp_paint_options_class_init (GimpPaintOptionsClass *klass)
|
||||
-180.0, 180.0, DEFAULT_BRUSH_ANGLE,
|
||||
GIMP_PARAM_STATIC_STRINGS);
|
||||
|
||||
GIMP_CONFIG_INSTALL_PROP_DOUBLE (object_class, PROP_BRUSH_SPACING,
|
||||
"brush-spacing", _("Brush Spacing"),
|
||||
1.0, 5000.0, DEFAULT_BRUSH_SPACING,
|
||||
GIMP_PARAM_STATIC_STRINGS);
|
||||
|
||||
GIMP_CONFIG_INSTALL_PROP_ENUM (object_class, PROP_APPLICATION_MODE,
|
||||
"application-mode", _("Every stamp has its own opacity"),
|
||||
GIMP_TYPE_PAINT_APPLICATION_MODE,
|
||||
@ -352,10 +352,6 @@ gimp_paint_options_set_property (GObject *object,
|
||||
options->brush_size = g_value_get_double (value);
|
||||
break;
|
||||
|
||||
case PROP_BRUSH_SPACING:
|
||||
options->brush_spacing = g_value_get_double (value);
|
||||
break;
|
||||
|
||||
case PROP_BRUSH_ASPECT_RATIO:
|
||||
options->brush_aspect_ratio = g_value_get_double (value);
|
||||
break;
|
||||
@ -364,6 +360,10 @@ gimp_paint_options_set_property (GObject *object,
|
||||
options->brush_angle = - 1.0 * g_value_get_double (value) / 360.0; /* let's make the angle mathematically correct */
|
||||
break;
|
||||
|
||||
case PROP_BRUSH_SPACING:
|
||||
options->brush_spacing = g_value_get_double (value);
|
||||
break;
|
||||
|
||||
case PROP_APPLICATION_MODE:
|
||||
options->application_mode = g_value_get_enum (value);
|
||||
break;
|
||||
@ -480,10 +480,6 @@ gimp_paint_options_get_property (GObject *object,
|
||||
g_value_set_double (value, options->brush_size);
|
||||
break;
|
||||
|
||||
case PROP_BRUSH_SPACING:
|
||||
g_value_set_double (value, options->brush_spacing);
|
||||
break;
|
||||
|
||||
case PROP_BRUSH_ASPECT_RATIO:
|
||||
g_value_set_double (value, options->brush_aspect_ratio);
|
||||
break;
|
||||
@ -492,6 +488,10 @@ gimp_paint_options_get_property (GObject *object,
|
||||
g_value_set_double (value, - 1.0 * options->brush_angle * 360.0); /* mathematically correct -> intuitively correct */
|
||||
break;
|
||||
|
||||
case PROP_BRUSH_SPACING:
|
||||
g_value_set_double (value, options->brush_spacing);
|
||||
break;
|
||||
|
||||
case PROP_APPLICATION_MODE:
|
||||
g_value_set_enum (value, options->application_mode);
|
||||
break;
|
||||
|
@ -83,9 +83,9 @@ struct _GimpPaintOptions
|
||||
gboolean use_applicator;
|
||||
|
||||
gdouble brush_size;
|
||||
gdouble brush_spacing;
|
||||
gdouble brush_angle;
|
||||
gdouble brush_aspect_ratio;
|
||||
gdouble brush_spacing;
|
||||
|
||||
GimpPaintApplicationMode application_mode;
|
||||
GimpPaintApplicationMode application_mode_save;
|
||||
|
@ -52,14 +52,14 @@
|
||||
|
||||
static void gimp_paint_options_gui_reset_size (GtkWidget *button,
|
||||
GimpPaintOptions *paint_options);
|
||||
static void gimp_paint_options_gui_reset_spacing
|
||||
(GtkWidget *button,
|
||||
GimpPaintOptions *paint_options);
|
||||
static void gimp_paint_options_gui_reset_aspect_ratio
|
||||
(GtkWidget *button,
|
||||
GimpPaintOptions *paint_options);
|
||||
static void gimp_paint_options_gui_reset_angle (GtkWidget *button,
|
||||
GimpPaintOptions *paint_options);
|
||||
static void gimp_paint_options_gui_reset_spacing
|
||||
(GtkWidget *button,
|
||||
GimpPaintOptions *paint_options);
|
||||
|
||||
static GtkWidget * dynamics_options_gui (GimpPaintOptions *paint_options,
|
||||
GType tool_type);
|
||||
@ -162,32 +162,6 @@ gimp_paint_options_gui (GimpToolOptions *tool_options)
|
||||
gtk_box_pack_start (GTK_BOX (vbox), hbox, FALSE, FALSE, 0);
|
||||
gtk_widget_show (hbox);
|
||||
|
||||
scale = gimp_prop_spin_scale_new (config, "brush-spacing",
|
||||
_("Spacing"),
|
||||
1.0, 10.0, 2);
|
||||
gimp_spin_scale_set_scale_limits (GIMP_SPIN_SCALE (scale), 1.0, 200.0);
|
||||
gimp_spin_scale_set_gamma (GIMP_SPIN_SCALE (scale), 1.7);
|
||||
gtk_box_pack_start (GTK_BOX (hbox), scale, TRUE, TRUE, 0);
|
||||
gtk_widget_show (scale);
|
||||
|
||||
button = gimp_icon_button_new (GIMP_STOCK_RESET, NULL);
|
||||
gtk_button_set_relief (GTK_BUTTON (button), GTK_RELIEF_NONE);
|
||||
gtk_image_set_from_icon_name (GTK_IMAGE (gtk_bin_get_child (GTK_BIN (button))),
|
||||
GIMP_STOCK_RESET, GTK_ICON_SIZE_MENU);
|
||||
gtk_box_pack_start (GTK_BOX (hbox), button, FALSE, FALSE, 0);
|
||||
gtk_widget_show (button);
|
||||
|
||||
g_signal_connect (button, "clicked",
|
||||
G_CALLBACK (gimp_paint_options_gui_reset_spacing),
|
||||
options);
|
||||
|
||||
gimp_help_set_help_data (button,
|
||||
_("Reset size to brush's native spacing"), NULL);
|
||||
|
||||
hbox = gtk_box_new (GTK_ORIENTATION_HORIZONTAL, 2);
|
||||
gtk_box_pack_start (GTK_BOX (vbox), hbox, FALSE, FALSE, 0);
|
||||
gtk_widget_show (hbox);
|
||||
|
||||
scale = gimp_prop_spin_scale_new (config, "brush-aspect-ratio",
|
||||
_("Aspect Ratio"),
|
||||
0.1, 1.0, 2);
|
||||
@ -232,6 +206,32 @@ gimp_paint_options_gui (GimpToolOptions *tool_options)
|
||||
gimp_help_set_help_data (button,
|
||||
_("Reset angle to zero"), NULL);
|
||||
|
||||
hbox = gtk_box_new (GTK_ORIENTATION_HORIZONTAL, 2);
|
||||
gtk_box_pack_start (GTK_BOX (vbox), hbox, FALSE, FALSE, 0);
|
||||
gtk_widget_show (hbox);
|
||||
|
||||
scale = gimp_prop_spin_scale_new (config, "brush-spacing",
|
||||
_("Spacing"),
|
||||
1.0, 10.0, 2);
|
||||
gimp_spin_scale_set_scale_limits (GIMP_SPIN_SCALE (scale), 1.0, 200.0);
|
||||
gimp_spin_scale_set_gamma (GIMP_SPIN_SCALE (scale), 1.7);
|
||||
gtk_box_pack_start (GTK_BOX (hbox), scale, TRUE, TRUE, 0);
|
||||
gtk_widget_show (scale);
|
||||
|
||||
button = gimp_icon_button_new (GIMP_STOCK_RESET, NULL);
|
||||
gtk_button_set_relief (GTK_BUTTON (button), GTK_RELIEF_NONE);
|
||||
gtk_image_set_from_icon_name (GTK_IMAGE (gtk_bin_get_child (GTK_BIN (button))),
|
||||
GIMP_STOCK_RESET, GTK_ICON_SIZE_MENU);
|
||||
gtk_box_pack_start (GTK_BOX (hbox), button, FALSE, FALSE, 0);
|
||||
gtk_widget_show (button);
|
||||
|
||||
g_signal_connect (button, "clicked",
|
||||
G_CALLBACK (gimp_paint_options_gui_reset_spacing),
|
||||
options);
|
||||
|
||||
gimp_help_set_help_data (button,
|
||||
_("Reset spacing to brush's native spacing"), NULL);
|
||||
|
||||
button = gimp_prop_dynamics_box_new (NULL, GIMP_CONTEXT (tool_options),
|
||||
_("Dynamics"), 2,
|
||||
"dynamics-view-type",
|
||||
@ -434,36 +434,36 @@ static void
|
||||
gimp_paint_options_gui_reset_size (GtkWidget *button,
|
||||
GimpPaintOptions *paint_options)
|
||||
{
|
||||
GimpBrush *brush = gimp_context_get_brush (GIMP_CONTEXT (paint_options));
|
||||
GimpBrush *brush = gimp_context_get_brush (GIMP_CONTEXT (paint_options));
|
||||
|
||||
if (brush)
|
||||
gimp_paint_options_set_default_brush_size (paint_options, brush);
|
||||
}
|
||||
|
||||
static void
|
||||
gimp_paint_options_gui_reset_spacing (GtkWidget *button,
|
||||
GimpPaintOptions *paint_options)
|
||||
{
|
||||
GimpBrush *brush = gimp_context_get_brush (GIMP_CONTEXT (paint_options));
|
||||
|
||||
if (brush)
|
||||
gimp_paint_options_set_default_brush_spacing (paint_options, brush);
|
||||
if (brush)
|
||||
gimp_paint_options_set_default_brush_size (paint_options, brush);
|
||||
}
|
||||
|
||||
static void
|
||||
gimp_paint_options_gui_reset_aspect_ratio (GtkWidget *button,
|
||||
GimpPaintOptions *paint_options)
|
||||
{
|
||||
g_object_set (paint_options,
|
||||
"brush-aspect-ratio", 0.0,
|
||||
NULL);
|
||||
g_object_set (paint_options,
|
||||
"brush-aspect-ratio", 0.0,
|
||||
NULL);
|
||||
}
|
||||
|
||||
static void
|
||||
gimp_paint_options_gui_reset_angle (GtkWidget *button,
|
||||
GimpPaintOptions *paint_options)
|
||||
{
|
||||
g_object_set (paint_options,
|
||||
"brush-angle", 0.0,
|
||||
NULL);
|
||||
g_object_set (paint_options,
|
||||
"brush-angle", 0.0,
|
||||
NULL);
|
||||
}
|
||||
|
||||
static void
|
||||
gimp_paint_options_gui_reset_spacing (GtkWidget *button,
|
||||
GimpPaintOptions *paint_options)
|
||||
{
|
||||
GimpBrush *brush = gimp_context_get_brush (GIMP_CONTEXT (paint_options));
|
||||
|
||||
if (brush)
|
||||
gimp_paint_options_set_default_brush_spacing (paint_options, brush);
|
||||
}
|
||||
|
Reference in New Issue
Block a user