app: add an edit button the the tool options' brush etc. selectors

which makes getting to the editor dialogs much easier.
This commit is contained in:
Michael Natterer
2011-04-07 23:48:08 +02:00
parent 4a73ac6869
commit fcbc5eebeb
4 changed files with 117 additions and 33 deletions

View File

@ -211,7 +211,8 @@ gimp_blend_options_gui (GimpToolOptions *tool_options)
_("Gradient"), 2, _("Gradient"), 2,
"gradient-view-type", "gradient-view-type",
"gradient-view-size", "gradient-view-size",
"gradient-reverse"); "gradient-reverse",
"gimp-gradient-editor");
gtk_box_pack_start (GTK_BOX (vbox), button, FALSE, FALSE, 0); gtk_box_pack_start (GTK_BOX (vbox), button, FALSE, FALSE, 0);
gtk_widget_show (button); gtk_widget_show (button);

View File

@ -121,7 +121,8 @@ gimp_paint_options_gui (GimpToolOptions *tool_options)
button = gimp_prop_brush_box_new (NULL, GIMP_CONTEXT (tool_options), button = gimp_prop_brush_box_new (NULL, GIMP_CONTEXT (tool_options),
_("Brush"), 2, _("Brush"), 2,
"brush-view-type", "brush-view-size"); "brush-view-type", "brush-view-size",
"gimp-brush-editor");
gtk_box_pack_start (GTK_BOX (vbox), button, FALSE, FALSE, 0); gtk_box_pack_start (GTK_BOX (vbox), button, FALSE, FALSE, 0);
gtk_widget_show (button); gtk_widget_show (button);
@ -200,7 +201,8 @@ gimp_paint_options_gui (GimpToolOptions *tool_options)
button = gimp_prop_dynamics_box_new (NULL, GIMP_CONTEXT (tool_options), button = gimp_prop_dynamics_box_new (NULL, GIMP_CONTEXT (tool_options),
_("Dynamics"), 2, _("Dynamics"), 2,
"dynamics-view-type", "dynamics-view-type",
"dynamics-view-size"); "dynamics-view-size",
"gimp-dynamics-editor");
gtk_box_pack_start (GTK_BOX (vbox), button, FALSE, FALSE, 0); gtk_box_pack_start (GTK_BOX (vbox), button, FALSE, FALSE, 0);
gtk_widget_show (button); gtk_widget_show (button);
@ -335,7 +337,8 @@ dynamics_options_gui (GimpPaintOptions *paint_options,
_("Gradient"), 2, _("Gradient"), 2,
"gradient-view-type", "gradient-view-type",
"gradient-view-size", "gradient-view-size",
"gradient-reverse"); "gradient-reverse",
"gimp-gradient-editor");
gtk_container_add (GTK_CONTAINER (inner_frame), box); gtk_container_add (GTK_CONTAINER (inner_frame), box);
gtk_widget_show (box); gtk_widget_show (box);
} }

View File

@ -52,11 +52,14 @@ static GtkWidget * gimp_viewable_box_new (GimpContainer *container,
GimpViewSize view_size, GimpViewSize view_size,
const gchar *dialog_identifier, const gchar *dialog_identifier,
const gchar *dialog_stock_id, const gchar *dialog_stock_id,
const gchar *dialog_tooltip); const gchar *dialog_tooltip,
const gchar *editor_id);
static GtkWidget * view_props_connect (GtkWidget *box, static GtkWidget * view_props_connect (GtkWidget *box,
GimpContext *context, GimpContext *context,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop); const gchar *view_size_prop);
static void gimp_viewable_box_edit_clicked (GtkWidget *widget,
GimpViewableButton *button);
static void gimp_gradient_box_reverse_notify (GObject *object, static void gimp_gradient_box_reverse_notify (GObject *object,
GParamSpec *pspec, GParamSpec *pspec,
GimpView *view); GimpView *view);
@ -70,7 +73,8 @@ brush_box_new (GimpContainer *container,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
GimpViewType view_type, GimpViewType view_type,
GimpViewSize view_size) GimpViewSize view_size,
const gchar *editor_id)
{ {
if (! container) if (! container)
container = gimp_data_factory_get_container (context->gimp->brush_factory); container = gimp_data_factory_get_container (context->gimp->brush_factory);
@ -79,7 +83,8 @@ brush_box_new (GimpContainer *container,
view_type, GIMP_VIEW_SIZE_SMALL, view_size, view_type, GIMP_VIEW_SIZE_SMALL, view_size,
"gimp-brush-grid|gimp-brush-list", "gimp-brush-grid|gimp-brush-list",
GIMP_STOCK_BRUSH, GIMP_STOCK_BRUSH,
_("Open the brush selection dialog")); _("Open the brush selection dialog"),
editor_id);
} }
GtkWidget * GtkWidget *
@ -93,7 +98,8 @@ gimp_brush_box_new (GimpContainer *container,
g_return_val_if_fail (GIMP_IS_CONTEXT (context), NULL); g_return_val_if_fail (GIMP_IS_CONTEXT (context), NULL);
return brush_box_new (container, context, label, spacing, return brush_box_new (container, context, label, spacing,
GIMP_VIEW_TYPE_GRID, GIMP_VIEW_SIZE_SMALL); GIMP_VIEW_TYPE_GRID, GIMP_VIEW_SIZE_SMALL,
NULL);
} }
GtkWidget * GtkWidget *
@ -102,7 +108,8 @@ gimp_prop_brush_box_new (GimpContainer *container,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop) const gchar *view_size_prop,
const gchar *editor_id)
{ {
GimpViewType view_type; GimpViewType view_type;
GimpViewSize view_size; GimpViewSize view_size;
@ -117,7 +124,8 @@ gimp_prop_brush_box_new (GimpContainer *container,
NULL); NULL);
return view_props_connect (brush_box_new (container, context, label, spacing, return view_props_connect (brush_box_new (container, context, label, spacing,
view_type, view_size), view_type, view_size,
editor_id),
context, context,
view_type_prop, view_size_prop); view_type_prop, view_size_prop);
} }
@ -129,7 +137,8 @@ dynamics_box_new (GimpContainer *container,
GimpContext *context, GimpContext *context,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
GimpViewSize view_size) GimpViewSize view_size,
const gchar *editor_id)
{ {
if (! container) if (! container)
container = gimp_data_factory_get_container (context->gimp->dynamics_factory); container = gimp_data_factory_get_container (context->gimp->dynamics_factory);
@ -138,7 +147,8 @@ dynamics_box_new (GimpContainer *container,
GIMP_VIEW_TYPE_LIST, GIMP_VIEW_SIZE_SMALL, view_size, GIMP_VIEW_TYPE_LIST, GIMP_VIEW_SIZE_SMALL, view_size,
"gimp-dynamics-list", "gimp-dynamics-list",
GIMP_STOCK_DYNAMICS, GIMP_STOCK_DYNAMICS,
_("Open the dynamics selection dialog")); _("Open the dynamics selection dialog"),
editor_id);
} }
GtkWidget * GtkWidget *
@ -152,7 +162,8 @@ gimp_dynamics_box_new (GimpContainer *container,
g_return_val_if_fail (GIMP_IS_CONTEXT (context), NULL); g_return_val_if_fail (GIMP_IS_CONTEXT (context), NULL);
return dynamics_box_new (container, context, label, spacing, return dynamics_box_new (container, context, label, spacing,
GIMP_VIEW_SIZE_SMALL); GIMP_VIEW_SIZE_SMALL,
NULL);
} }
GtkWidget * GtkWidget *
@ -161,7 +172,8 @@ gimp_prop_dynamics_box_new (GimpContainer *container,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop) const gchar *view_size_prop,
const gchar *editor_id)
{ {
GimpViewType view_type; GimpViewType view_type;
GimpViewSize view_size; GimpViewSize view_size;
@ -176,7 +188,8 @@ gimp_prop_dynamics_box_new (GimpContainer *container,
NULL); NULL);
return view_props_connect (dynamics_box_new (container, context, label, return view_props_connect (dynamics_box_new (container, context, label,
spacing, view_size), spacing, view_size,
editor_id),
context, context,
view_type_prop, view_size_prop); view_type_prop, view_size_prop);
} }
@ -199,7 +212,8 @@ pattern_box_new (GimpContainer *container,
view_type, GIMP_VIEW_SIZE_SMALL, view_size, view_type, GIMP_VIEW_SIZE_SMALL, view_size,
"gimp-pattern-grid|gimp-pattern-list", "gimp-pattern-grid|gimp-pattern-list",
GIMP_STOCK_PATTERN, GIMP_STOCK_PATTERN,
_("Open the pattern selection dialog")); _("Open the pattern selection dialog"),
NULL);
} }
GtkWidget * GtkWidget *
@ -252,7 +266,8 @@ gradient_box_new (GimpContainer *container,
gint spacing, gint spacing,
GimpViewType view_type, GimpViewType view_type,
GimpViewSize view_size, GimpViewSize view_size,
const gchar *reverse_prop) const gchar *reverse_prop,
const gchar *editor_id)
{ {
GtkWidget *hbox; GtkWidget *hbox;
GtkWidget *button; GtkWidget *button;
@ -265,7 +280,8 @@ gradient_box_new (GimpContainer *container,
view_type, GIMP_VIEW_SIZE_LARGE, view_size, view_type, GIMP_VIEW_SIZE_LARGE, view_size,
"gimp-gradient-list|gimp-gradient-grid", "gimp-gradient-list|gimp-gradient-grid",
GIMP_STOCK_GRADIENT, GIMP_STOCK_GRADIENT,
_("Open the gradient selection dialog")); _("Open the gradient selection dialog"),
editor_id);
children = gtk_container_get_children (GTK_CONTAINER (hbox)); children = gtk_container_get_children (GTK_CONTAINER (hbox));
button = children->data; button = children->data;
@ -275,20 +291,27 @@ gradient_box_new (GimpContainer *container,
if (reverse_prop) if (reverse_prop)
{ {
GtkWidget *vbox;
GtkWidget *toggle; GtkWidget *toggle;
GtkWidget *view; GtkWidget *view;
GtkWidget *image; GtkWidget *image;
gchar *signal_name; gchar *signal_name;
vbox = gtk_vbox_new (FALSE, 0);
gtk_box_pack_start (GTK_BOX (hbox), vbox, FALSE, FALSE, 0);
gtk_widget_show (vbox);
toggle = gimp_prop_check_button_new (G_OBJECT (context), reverse_prop, toggle = gimp_prop_check_button_new (G_OBJECT (context), reverse_prop,
NULL); NULL);
gtk_box_pack_start (GTK_BOX (hbox), toggle, FALSE, FALSE, 0); gtk_toggle_button_set_mode (GTK_TOGGLE_BUTTON (toggle), FALSE);
gtk_box_pack_end (GTK_BOX (vbox), toggle, FALSE, FALSE, 0);
gtk_widget_show (toggle); gtk_widget_show (toggle);
gimp_help_set_help_data (toggle, _("Reverse"), NULL); gimp_help_set_help_data (toggle, _("Reverse"), NULL);
image = gtk_image_new_from_stock (GIMP_STOCK_FLIP_HORIZONTAL, image = gtk_image_new_from_stock (GIMP_STOCK_FLIP_HORIZONTAL,
GTK_ICON_SIZE_MENU); GTK_ICON_SIZE_MENU);
gtk_misc_set_alignment (GTK_MISC (image), 0.5, 1.0);
gtk_container_add (GTK_CONTAINER (toggle), image); gtk_container_add (GTK_CONTAINER (toggle), image);
gtk_widget_show (image); gtk_widget_show (image);
@ -321,7 +344,8 @@ gimp_gradient_box_new (GimpContainer *container,
return gradient_box_new (container, context, label, spacing, return gradient_box_new (container, context, label, spacing,
GIMP_VIEW_TYPE_LIST, GIMP_VIEW_SIZE_LARGE, GIMP_VIEW_TYPE_LIST, GIMP_VIEW_SIZE_LARGE,
reverse_prop); reverse_prop,
NULL);
} }
GtkWidget * GtkWidget *
@ -331,7 +355,8 @@ gimp_prop_gradient_box_new (GimpContainer *container,
gint spacing, gint spacing,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop, const gchar *view_size_prop,
const gchar *reverse_prop) const gchar *reverse_prop,
const gchar *editor_id)
{ {
GimpViewType view_type; GimpViewType view_type;
GimpViewSize view_size; GimpViewSize view_size;
@ -347,7 +372,8 @@ gimp_prop_gradient_box_new (GimpContainer *container,
return view_props_connect (gradient_box_new (container, context, label, spacing, return view_props_connect (gradient_box_new (container, context, label, spacing,
view_type, view_size, view_type, view_size,
reverse_prop), reverse_prop,
editor_id),
context, context,
view_type_prop, view_size_prop); view_type_prop, view_size_prop);
} }
@ -361,7 +387,8 @@ palette_box_new (GimpContainer *container,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
GimpViewType view_type, GimpViewType view_type,
GimpViewSize view_size) GimpViewSize view_size,
const gchar *editor_id)
{ {
if (! container) if (! container)
container = gimp_data_factory_get_container (context->gimp->palette_factory); container = gimp_data_factory_get_container (context->gimp->palette_factory);
@ -370,7 +397,8 @@ palette_box_new (GimpContainer *container,
view_type, GIMP_VIEW_SIZE_MEDIUM, view_size, view_type, GIMP_VIEW_SIZE_MEDIUM, view_size,
"gimp-palette-list|gimp-palette-grid", "gimp-palette-list|gimp-palette-grid",
GIMP_STOCK_PALETTE, GIMP_STOCK_PALETTE,
_("Open the palette selection dialog")); _("Open the palette selection dialog"),
editor_id);
} }
GtkWidget * GtkWidget *
@ -384,7 +412,8 @@ gimp_palette_box_new (GimpContainer *container,
g_return_val_if_fail (GIMP_IS_CONTEXT (context), NULL); g_return_val_if_fail (GIMP_IS_CONTEXT (context), NULL);
return palette_box_new (container, context, label, spacing, return palette_box_new (container, context, label, spacing,
GIMP_VIEW_TYPE_LIST, GIMP_VIEW_SIZE_MEDIUM); GIMP_VIEW_TYPE_LIST, GIMP_VIEW_SIZE_MEDIUM,
NULL);
} }
GtkWidget * GtkWidget *
@ -393,7 +422,8 @@ gimp_prop_palette_box_new (GimpContainer *container,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop) const gchar *view_size_prop,
const gchar *editor_id)
{ {
GimpViewType view_type; GimpViewType view_type;
GimpViewSize view_size; GimpViewSize view_size;
@ -408,7 +438,8 @@ gimp_prop_palette_box_new (GimpContainer *container,
NULL); NULL);
return view_props_connect (palette_box_new (container, context, label, spacing, return view_props_connect (palette_box_new (container, context, label, spacing,
view_type, view_size), view_type, view_size,
editor_id),
context, context,
view_type_prop, view_size_prop); view_type_prop, view_size_prop);
} }
@ -431,7 +462,8 @@ font_box_new (GimpContainer *container,
view_type, GIMP_VIEW_SIZE_SMALL, view_size, view_type, GIMP_VIEW_SIZE_SMALL, view_size,
"gimp-font-list|gimp-font-grid", "gimp-font-list|gimp-font-grid",
GIMP_STOCK_FONT, GIMP_STOCK_FONT,
_("Open the font selection dialog")); _("Open the font selection dialog"),
NULL);
} }
GtkWidget * GtkWidget *
@ -487,7 +519,8 @@ gimp_viewable_box_new (GimpContainer *container,
GimpViewSize view_size, GimpViewSize view_size,
const gchar *dialog_identifier, const gchar *dialog_identifier,
const gchar *dialog_stock_id, const gchar *dialog_stock_id,
const gchar *dialog_tooltip) const gchar *dialog_tooltip,
const gchar *editor_id)
{ {
GtkWidget *hbox; GtkWidget *hbox;
GtkWidget *button; GtkWidget *button;
@ -530,6 +563,37 @@ gimp_viewable_box_new (GimpContainer *container,
gtk_box_pack_start (GTK_BOX (vbox), entry, label ? FALSE: TRUE, FALSE, 0); gtk_box_pack_start (GTK_BOX (vbox), entry, label ? FALSE: TRUE, FALSE, 0);
gtk_widget_show (entry); gtk_widget_show (entry);
if (editor_id)
{
GtkWidget *edit_vbox;
GtkWidget *edit_button;
GtkWidget *image;
edit_vbox = gtk_vbox_new (FALSE, 0);
gtk_box_pack_end (GTK_BOX (hbox), edit_vbox, FALSE, FALSE, 0);
gtk_widget_show (edit_vbox);
edit_button = gtk_button_new ();
gtk_button_set_relief (GTK_BUTTON (edit_button), GTK_RELIEF_NONE);
gtk_box_pack_end (GTK_BOX (edit_vbox), edit_button, FALSE, FALSE, 0);
gtk_widget_show (edit_button);
image = gtk_image_new_from_stock (GIMP_STOCK_EDIT,
GTK_ICON_SIZE_BUTTON);
gtk_misc_set_alignment (GTK_MISC (image), 0.5, 1.0);
gtk_container_add (GTK_CONTAINER (edit_button), image);
gtk_widget_show (image);
g_object_set_data_full (G_OBJECT (button),
"gimp-viewable-box-editor",
g_strdup (editor_id),
(GDestroyNotify) g_free);
g_signal_connect (edit_button, "clicked",
G_CALLBACK (gimp_viewable_box_edit_clicked),
button);
}
return hbox; return hbox;
} }
@ -549,6 +613,18 @@ view_props_connect (GtkWidget *box,
return box; return box;
} }
static void
gimp_viewable_box_edit_clicked (GtkWidget *widget,
GimpViewableButton *button)
{
const gchar *editor_id = g_object_get_data (G_OBJECT (button),
"gimp-viewable-box-editor");
gimp_dialog_factory_dialog_raise (button->dialog_factory,
gtk_widget_get_screen (widget),
editor_id, -1);
}
static void static void
gimp_gradient_box_reverse_notify (GObject *object, gimp_gradient_box_reverse_notify (GObject *object,
GParamSpec *pspec, GParamSpec *pspec,

View File

@ -28,7 +28,8 @@ GtkWidget * gimp_prop_brush_box_new (GimpContainer *container,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop); const gchar *view_size_prop,
const gchar *editor_id);
GtkWidget * gimp_dynamics_box_new (GimpContainer *container, GtkWidget * gimp_dynamics_box_new (GimpContainer *container,
GimpContext *context, GimpContext *context,
const gchar *label, const gchar *label,
@ -38,7 +39,8 @@ GtkWidget * gimp_prop_dynamics_box_new (GimpContainer *container,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop); const gchar *view_size_prop,
const gchar *editor_id);
GtkWidget * gimp_pattern_box_new (GimpContainer *container, GtkWidget * gimp_pattern_box_new (GimpContainer *container,
GimpContext *context, GimpContext *context,
@ -62,7 +64,8 @@ GtkWidget * gimp_prop_gradient_box_new (GimpContainer *container,
gint scacing, gint scacing,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop, const gchar *view_size_prop,
const gchar *reverse_prop); const gchar *reverse_prop,
const gchar *editor_id);
GtkWidget * gimp_palette_box_new (GimpContainer *container, GtkWidget * gimp_palette_box_new (GimpContainer *container,
GimpContext *context, GimpContext *context,
@ -73,7 +76,8 @@ GtkWidget * gimp_prop_palette_box_new (GimpContainer *container,
const gchar *label, const gchar *label,
gint spacing, gint spacing,
const gchar *view_type_prop, const gchar *view_type_prop,
const gchar *view_size_prop); const gchar *view_size_prop,
const gchar *editor_id);
GtkWidget * gimp_font_box_new (GimpContainer *container, GtkWidget * gimp_font_box_new (GimpContainer *container,
GimpContext *context, GimpContext *context,