colorbutton: Drop the Private struct

This commit is contained in:
Matthias Clasen 2020-04-13 00:28:11 -04:00
parent 15d58a868a
commit 7a0355fcb1

View File

@ -66,16 +66,7 @@ typedef struct _GtkColorButtonClass GtkColorButtonClass;
struct _GtkColorButton { struct _GtkColorButton {
GtkWidget parent_instance; GtkWidget parent_instance;
};
struct _GtkColorButtonClass {
GtkWidgetClass parent_class;
void (* color_set) (GtkColorButton *cp);
};
typedef struct
{
GtkWidget *button; GtkWidget *button;
GtkWidget *swatch; /* Widget where we draw the color sample */ GtkWidget *swatch; /* Widget where we draw the color sample */
@ -87,7 +78,13 @@ typedef struct
guint use_alpha : 1; /* Use alpha or not */ guint use_alpha : 1; /* Use alpha or not */
guint show_editor : 1; guint show_editor : 1;
guint modal : 1; guint modal : 1;
} GtkColorButtonPrivate; };
struct _GtkColorButtonClass {
GtkWidgetClass parent_class;
void (* color_set) (GtkColorButton *cp);
};
/* Properties */ /* Properties */
enum enum
@ -128,7 +125,6 @@ static guint color_button_signals[LAST_SIGNAL] = { 0 };
static void gtk_color_button_iface_init (GtkColorChooserInterface *iface); static void gtk_color_button_iface_init (GtkColorChooserInterface *iface);
G_DEFINE_TYPE_WITH_CODE (GtkColorButton, gtk_color_button, GTK_TYPE_WIDGET, G_DEFINE_TYPE_WITH_CODE (GtkColorButton, gtk_color_button, GTK_TYPE_WIDGET,
G_ADD_PRIVATE (GtkColorButton)
G_IMPLEMENT_INTERFACE (GTK_TYPE_COLOR_CHOOSER, G_IMPLEMENT_INTERFACE (GTK_TYPE_COLOR_CHOOSER,
gtk_color_button_iface_init)) gtk_color_button_iface_init))
@ -259,44 +255,41 @@ gtk_color_button_drag_prepare (GtkDragSource *source,
double y, double y,
GtkColorButton *button) GtkColorButton *button)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button); return gdk_content_provider_new_typed (GDK_TYPE_RGBA, &button->rgba);
return gdk_content_provider_new_typed (GDK_TYPE_RGBA, &priv->rgba);
} }
static void static void
gtk_color_button_init (GtkColorButton *button) gtk_color_button_init (GtkColorButton *button)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
PangoLayout *layout; PangoLayout *layout;
PangoRectangle rect; PangoRectangle rect;
GtkDragSource *source; GtkDragSource *source;
GtkDropTarget *dest; GtkDropTarget *dest;
priv->button = gtk_button_new (); button->button = gtk_button_new ();
g_signal_connect (priv->button, "clicked", G_CALLBACK (gtk_color_button_clicked), button); g_signal_connect (button->button, "clicked", G_CALLBACK (gtk_color_button_clicked), button);
gtk_widget_set_parent (priv->button, GTK_WIDGET (button)); gtk_widget_set_parent (button->button, GTK_WIDGET (button));
priv->swatch = gtk_color_swatch_new (); button->swatch = gtk_color_swatch_new ();
gtk_widget_set_can_focus (priv->swatch, FALSE); gtk_widget_set_can_focus (button->swatch, FALSE);
g_object_set (priv->swatch, "has-menu", FALSE, NULL); g_object_set (button->swatch, "has-menu", FALSE, NULL);
layout = gtk_widget_create_pango_layout (GTK_WIDGET (button), "Black"); layout = gtk_widget_create_pango_layout (GTK_WIDGET (button), "Black");
pango_layout_get_pixel_extents (layout, NULL, &rect); pango_layout_get_pixel_extents (layout, NULL, &rect);
g_object_unref (layout); g_object_unref (layout);
gtk_widget_set_size_request (priv->swatch, rect.width, rect.height); gtk_widget_set_size_request (button->swatch, rect.width, rect.height);
gtk_container_add (GTK_CONTAINER (priv->button), priv->swatch); gtk_container_add (GTK_CONTAINER (button->button), button->swatch);
priv->title = g_strdup (_("Pick a Color")); /* default title */ button->title = g_strdup (_("Pick a Color")); /* default title */
/* Start with opaque black, alpha disabled */ /* Start with opaque black, alpha disabled */
priv->rgba.red = 0; button->rgba.red = 0;
priv->rgba.green = 0; button->rgba.green = 0;
priv->rgba.blue = 0; button->rgba.blue = 0;
priv->rgba.alpha = 1; button->rgba.alpha = 1;
priv->use_alpha = FALSE; button->use_alpha = FALSE;
priv->modal = TRUE; button->modal = TRUE;
dest = gtk_drop_target_new (GDK_TYPE_RGBA, GDK_ACTION_COPY); dest = gtk_drop_target_new (GDK_TYPE_RGBA, GDK_ACTION_COPY);
g_signal_connect (dest, "drop", G_CALLBACK (gtk_color_button_drop), button); g_signal_connect (dest, "drop", G_CALLBACK (gtk_color_button_drop), button);
@ -305,22 +298,21 @@ gtk_color_button_init (GtkColorButton *button)
source = gtk_drag_source_new (); source = gtk_drag_source_new ();
g_signal_connect (source, "prepare", G_CALLBACK (gtk_color_button_drag_prepare), button); g_signal_connect (source, "prepare", G_CALLBACK (gtk_color_button_drag_prepare), button);
gtk_event_controller_set_propagation_phase (GTK_EVENT_CONTROLLER (source), GTK_PHASE_CAPTURE); gtk_event_controller_set_propagation_phase (GTK_EVENT_CONTROLLER (source), GTK_PHASE_CAPTURE);
gtk_widget_add_controller (priv->button, GTK_EVENT_CONTROLLER (source)); gtk_widget_add_controller (button->button, GTK_EVENT_CONTROLLER (source));
gtk_widget_add_css_class (priv->button, "color"); gtk_widget_add_css_class (button->button, "color");
} }
static void static void
gtk_color_button_finalize (GObject *object) gtk_color_button_finalize (GObject *object)
{ {
GtkColorButton *button = GTK_COLOR_BUTTON (object); GtkColorButton *button = GTK_COLOR_BUTTON (object);
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
if (priv->cs_dialog != NULL) if (button->cs_dialog != NULL)
gtk_widget_destroy (priv->cs_dialog); gtk_widget_destroy (button->cs_dialog);
g_free (priv->title); g_free (button->title);
gtk_widget_unparent (priv->button); gtk_widget_unparent (button->button);
G_OBJECT_CLASS (gtk_color_button_parent_class)->finalize (object); G_OBJECT_CLASS (gtk_color_button_parent_class)->finalize (object);
} }
@ -364,9 +356,8 @@ dialog_destroy (GtkWidget *widget,
gpointer data) gpointer data)
{ {
GtkColorButton *button = GTK_COLOR_BUTTON (data); GtkColorButton *button = GTK_COLOR_BUTTON (data);
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
priv->cs_dialog = NULL; button->cs_dialog = NULL;
return FALSE; return FALSE;
} }
@ -381,10 +372,9 @@ dialog_response (GtkDialog *dialog,
else if (response == GTK_RESPONSE_OK) else if (response == GTK_RESPONSE_OK)
{ {
GtkColorButton *button = GTK_COLOR_BUTTON (data); GtkColorButton *button = GTK_COLOR_BUTTON (data);
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
gtk_color_chooser_get_rgba (GTK_COLOR_CHOOSER (dialog), &priv->rgba); gtk_color_chooser_get_rgba (GTK_COLOR_CHOOSER (dialog), &button->rgba);
gtk_color_swatch_set_rgba (GTK_COLOR_SWATCH (priv->swatch), &priv->rgba); gtk_color_swatch_set_rgba (GTK_COLOR_SWATCH (button->swatch), &button->rgba);
gtk_widget_hide (GTK_WIDGET (dialog)); gtk_widget_hide (GTK_WIDGET (dialog));
@ -402,17 +392,16 @@ dialog_response (GtkDialog *dialog,
static void static void
ensure_dialog (GtkColorButton *button) ensure_dialog (GtkColorButton *button)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
GtkWidget *parent, *dialog; GtkWidget *parent, *dialog;
if (priv->cs_dialog != NULL) if (button->cs_dialog != NULL)
return; return;
parent = GTK_WIDGET (gtk_widget_get_root (GTK_WIDGET (button))); parent = GTK_WIDGET (gtk_widget_get_root (GTK_WIDGET (button)));
priv->cs_dialog = dialog = gtk_color_chooser_dialog_new (priv->title, NULL); button->cs_dialog = dialog = gtk_color_chooser_dialog_new (button->title, NULL);
gtk_window_set_hide_on_close (GTK_WINDOW (dialog), TRUE); gtk_window_set_hide_on_close (GTK_WINDOW (dialog), TRUE);
gtk_window_set_modal (GTK_WINDOW (dialog), priv->modal); gtk_window_set_modal (GTK_WINDOW (dialog), button->modal);
if (GTK_IS_WINDOW (parent)) if (GTK_IS_WINDOW (parent))
{ {
@ -435,19 +424,18 @@ gtk_color_button_clicked (GtkButton *b,
gpointer user_data) gpointer user_data)
{ {
GtkColorButton *button = user_data; GtkColorButton *button = user_data;
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
/* if dialog already exists, make sure it's shown and raised */ /* if dialog already exists, make sure it's shown and raised */
ensure_dialog (button); ensure_dialog (button);
g_object_set (priv->cs_dialog, "show-editor", priv->show_editor, NULL); g_object_set (button->cs_dialog, "show-editor", button->show_editor, NULL);
gtk_color_chooser_set_use_alpha (GTK_COLOR_CHOOSER (priv->cs_dialog), priv->use_alpha); gtk_color_chooser_set_use_alpha (GTK_COLOR_CHOOSER (button->cs_dialog), button->use_alpha);
gtk_color_chooser_set_rgba (GTK_COLOR_CHOOSER (priv->cs_dialog), &priv->rgba); gtk_color_chooser_set_rgba (GTK_COLOR_CHOOSER (button->cs_dialog), &button->rgba);
G_GNUC_BEGIN_IGNORE_DEPRECATIONS G_GNUC_BEGIN_IGNORE_DEPRECATIONS
gtk_window_present (GTK_WINDOW (priv->cs_dialog)); gtk_window_present (GTK_WINDOW (button->cs_dialog));
G_GNUC_END_IGNORE_DEPRECATIONS G_GNUC_END_IGNORE_DEPRECATIONS
} }
@ -455,13 +443,13 @@ static void
gtk_color_button_set_rgba (GtkColorChooser *chooser, gtk_color_button_set_rgba (GtkColorChooser *chooser,
const GdkRGBA *rgba) const GdkRGBA *rgba)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (GTK_COLOR_BUTTON (chooser)); GtkColorButton *button = GTK_COLOR_BUTTON (chooser);
g_return_if_fail (GTK_IS_COLOR_BUTTON (chooser)); g_return_if_fail (GTK_IS_COLOR_BUTTON (chooser));
g_return_if_fail (rgba != NULL); g_return_if_fail (rgba != NULL);
priv->rgba = *rgba; button->rgba = *rgba;
gtk_color_swatch_set_rgba (GTK_COLOR_SWATCH (priv->swatch), &priv->rgba); gtk_color_swatch_set_rgba (GTK_COLOR_SWATCH (button->swatch), &button->rgba);
g_object_notify (G_OBJECT (chooser), "rgba"); g_object_notify (G_OBJECT (chooser), "rgba");
} }
@ -470,27 +458,25 @@ static void
gtk_color_button_get_rgba (GtkColorChooser *chooser, gtk_color_button_get_rgba (GtkColorChooser *chooser,
GdkRGBA *rgba) GdkRGBA *rgba)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (GTK_COLOR_BUTTON (chooser)); GtkColorButton *button = GTK_COLOR_BUTTON (chooser);
g_return_if_fail (GTK_IS_COLOR_BUTTON (chooser)); g_return_if_fail (GTK_IS_COLOR_BUTTON (chooser));
g_return_if_fail (rgba != NULL); g_return_if_fail (rgba != NULL);
*rgba = priv->rgba; *rgba = button->rgba;
} }
static void static void
set_use_alpha (GtkColorButton *button, set_use_alpha (GtkColorButton *button,
gboolean use_alpha) gboolean use_alpha)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
use_alpha = (use_alpha != FALSE); use_alpha = (use_alpha != FALSE);
if (priv->use_alpha != use_alpha) if (button->use_alpha != use_alpha)
{ {
priv->use_alpha = use_alpha; button->use_alpha = use_alpha;
gtk_color_swatch_set_use_alpha (GTK_COLOR_SWATCH (priv->swatch), use_alpha); gtk_color_swatch_set_use_alpha (GTK_COLOR_SWATCH (button->swatch), use_alpha);
g_object_notify (G_OBJECT (button), "use-alpha"); g_object_notify (G_OBJECT (button), "use-alpha");
} }
@ -507,17 +493,16 @@ void
gtk_color_button_set_title (GtkColorButton *button, gtk_color_button_set_title (GtkColorButton *button,
const gchar *title) const gchar *title)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
gchar *old_title; gchar *old_title;
g_return_if_fail (GTK_IS_COLOR_BUTTON (button)); g_return_if_fail (GTK_IS_COLOR_BUTTON (button));
old_title = priv->title; old_title = button->title;
priv->title = g_strdup (title); button->title = g_strdup (title);
g_free (old_title); g_free (old_title);
if (priv->cs_dialog) if (button->cs_dialog)
gtk_window_set_title (GTK_WINDOW (priv->cs_dialog), priv->title); gtk_window_set_title (GTK_WINDOW (button->cs_dialog), button->title);
g_object_notify (G_OBJECT (button), "title"); g_object_notify (G_OBJECT (button), "title");
} }
@ -533,11 +518,9 @@ gtk_color_button_set_title (GtkColorButton *button,
const gchar * const gchar *
gtk_color_button_get_title (GtkColorButton *button) gtk_color_button_get_title (GtkColorButton *button)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
g_return_val_if_fail (GTK_IS_COLOR_BUTTON (button), NULL); g_return_val_if_fail (GTK_IS_COLOR_BUTTON (button), NULL);
return priv->title; return button->title;
} }
/** /**
@ -551,17 +534,15 @@ void
gtk_color_button_set_modal (GtkColorButton *button, gtk_color_button_set_modal (GtkColorButton *button,
gboolean modal) gboolean modal)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
g_return_if_fail (GTK_IS_COLOR_BUTTON (button)); g_return_if_fail (GTK_IS_COLOR_BUTTON (button));
if (priv->modal == modal) if (button->modal == modal)
return; return;
priv->modal = modal; button->modal = modal;
if (priv->cs_dialog) if (button->cs_dialog)
gtk_window_set_modal (GTK_WINDOW (priv->cs_dialog), priv->modal); gtk_window_set_modal (GTK_WINDOW (button->cs_dialog), button->modal);
g_object_notify (G_OBJECT (button), "modal"); g_object_notify (G_OBJECT (button), "modal");
} }
@ -577,11 +558,9 @@ gtk_color_button_set_modal (GtkColorButton *button,
gboolean gboolean
gtk_color_button_get_modal (GtkColorButton *button) gtk_color_button_get_modal (GtkColorButton *button)
{ {
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
g_return_val_if_fail (GTK_IS_COLOR_BUTTON (button), FALSE); g_return_val_if_fail (GTK_IS_COLOR_BUTTON (button), FALSE);
return priv->modal; return button->modal;
} }
static void static void
@ -591,7 +570,6 @@ gtk_color_button_set_property (GObject *object,
GParamSpec *pspec) GParamSpec *pspec)
{ {
GtkColorButton *button = GTK_COLOR_BUTTON (object); GtkColorButton *button = GTK_COLOR_BUTTON (object);
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
switch (param_id) switch (param_id)
{ {
@ -607,9 +585,9 @@ gtk_color_button_set_property (GObject *object,
case PROP_SHOW_EDITOR: case PROP_SHOW_EDITOR:
{ {
gboolean show_editor = g_value_get_boolean (value); gboolean show_editor = g_value_get_boolean (value);
if (priv->show_editor != show_editor) if (button->show_editor != show_editor)
{ {
priv->show_editor = show_editor; button->show_editor = show_editor;
g_object_notify (object, "show-editor"); g_object_notify (object, "show-editor");
} }
} }
@ -630,12 +608,11 @@ gtk_color_button_get_property (GObject *object,
GParamSpec *pspec) GParamSpec *pspec)
{ {
GtkColorButton *button = GTK_COLOR_BUTTON (object); GtkColorButton *button = GTK_COLOR_BUTTON (object);
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
switch (param_id) switch (param_id)
{ {
case PROP_USE_ALPHA: case PROP_USE_ALPHA:
g_value_set_boolean (value, priv->use_alpha); g_value_set_boolean (value, button->use_alpha);
break; break;
case PROP_TITLE: case PROP_TITLE:
g_value_set_string (value, gtk_color_button_get_title (button)); g_value_set_string (value, gtk_color_button_get_title (button));
@ -649,10 +626,10 @@ gtk_color_button_get_property (GObject *object,
} }
break; break;
case PROP_SHOW_EDITOR: case PROP_SHOW_EDITOR:
g_value_set_boolean (value, priv->show_editor); g_value_set_boolean (value, button->show_editor);
break; break;
case PROP_MODAL: case PROP_MODAL:
g_value_set_boolean (value, priv->modal); g_value_set_boolean (value, button->modal);
break; break;
default: default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec); G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
@ -668,11 +645,10 @@ gtk_color_button_add_palette (GtkColorChooser *chooser,
GdkRGBA *colors) GdkRGBA *colors)
{ {
GtkColorButton *button = GTK_COLOR_BUTTON (chooser); GtkColorButton *button = GTK_COLOR_BUTTON (chooser);
GtkColorButtonPrivate *priv = gtk_color_button_get_instance_private (button);
ensure_dialog (button); ensure_dialog (button);
gtk_color_chooser_add_palette (GTK_COLOR_CHOOSER (priv->cs_dialog), gtk_color_chooser_add_palette (GTK_COLOR_CHOOSER (button->cs_dialog),
orientation, colors_per_line, n_colors, colors); orientation, colors_per_line, n_colors, colors);
} }