diff options
author | Benjamin Otte <otte@redhat.com> | 2020-01-27 22:58:15 +0100 |
---|---|---|
committer | Benjamin Otte <otte@redhat.com> | 2020-01-28 02:17:31 +0100 |
commit | afbff574f5ab49c81f05d1283339763a11e307ec (patch) | |
tree | e3dde31a9cfb911b0181ce5761c7c094a0f9e3df /gtk | |
parent | aff25212df78271b71db543f4d61947f1d4b6d68 (diff) | |
download | gtk+-afbff574f5ab49c81f05d1283339763a11e307ec.tar.gz |
filechooser: Remove default size machinery
The FileChooser ToolKit (fctk) had its own machinery to handle default
sizes which was completely busted and trying to marshal random numbers
through the widget hierarchy that maybe made sense in 2012 but don't do
now.
Get rid of it, just keep the dialog's GSetting - which funnily enough
used to be written by the dialog but written by the widget.
But that's fctk for you.
Diffstat (limited to 'gtk')
-rw-r--r-- | gtk/gtkfilechooserdialog.c | 48 | ||||
-rw-r--r-- | gtk/gtkfilechooserembed.c | 42 | ||||
-rw-r--r-- | gtk/gtkfilechooserembed.h | 8 | ||||
-rw-r--r-- | gtk/gtkfilechooserwidget.c | 92 | ||||
-rw-r--r-- | gtk/gtkfontchooserwidget.c | 15 | ||||
-rw-r--r-- | gtk/ui/gtkfilechooserdialog.ui | 1 |
6 files changed, 25 insertions, 181 deletions
diff --git a/gtk/gtkfilechooserdialog.c b/gtk/gtkfilechooserdialog.c index 2ce649e258..b2929d988d 100644 --- a/gtk/gtkfilechooserdialog.c +++ b/gtk/gtkfilechooserdialog.c @@ -240,6 +240,7 @@ static void gtk_file_chooser_dialog_get_property (GObject *obj static void gtk_file_chooser_dialog_notify (GObject *object, GParamSpec *pspec); +static void gtk_file_chooser_dialog_realize (GtkWidget *widget); static void gtk_file_chooser_dialog_map (GtkWidget *widget); static void gtk_file_chooser_dialog_unmap (GtkWidget *widget); static void gtk_file_chooser_dialog_size_allocate (GtkWidget *widget, @@ -248,8 +249,6 @@ static void gtk_file_chooser_dialog_size_allocate (GtkWidget *wid int baseline); static void file_chooser_widget_file_activated (GtkFileChooser *chooser, GtkFileChooserDialog *dialog); -static void file_chooser_widget_default_size_changed (GtkWidget *widget, - GtkFileChooserDialog *dialog); static void file_chooser_widget_response_requested (GtkWidget *widget, GtkFileChooserDialog *dialog); static void file_chooser_widget_selection_changed (GtkWidget *widget, @@ -275,6 +274,7 @@ gtk_file_chooser_dialog_class_init (GtkFileChooserDialogClass *class) gobject_class->get_property = gtk_file_chooser_dialog_get_property; gobject_class->notify = gtk_file_chooser_dialog_notify; + widget_class->realize = gtk_file_chooser_dialog_realize; widget_class->map = gtk_file_chooser_dialog_map; widget_class->unmap = gtk_file_chooser_dialog_unmap; widget_class->size_allocate = gtk_file_chooser_dialog_size_allocate; @@ -292,7 +292,6 @@ gtk_file_chooser_dialog_class_init (GtkFileChooserDialogClass *class) gtk_widget_class_bind_template_child_private (widget_class, GtkFileChooserDialog, buttons); gtk_widget_class_bind_template_callback (widget_class, response_cb); gtk_widget_class_bind_template_callback (widget_class, file_chooser_widget_file_activated); - gtk_widget_class_bind_template_callback (widget_class, file_chooser_widget_default_size_changed); gtk_widget_class_bind_template_callback (widget_class, file_chooser_widget_response_requested); gtk_widget_class_bind_template_callback (widget_class, file_chooser_widget_selection_changed); } @@ -358,32 +357,6 @@ file_chooser_widget_file_activated (GtkFileChooser *chooser, } static void -file_chooser_widget_default_size_changed (GtkWidget *widget, - GtkFileChooserDialog *dialog) -{ - GtkFileChooserDialogPrivate *priv = gtk_file_chooser_dialog_get_instance_private (dialog); - gint default_width, default_height; - GtkRequisition req, widget_req; - - /* Unset any previously set size */ - gtk_widget_set_size_request (GTK_WIDGET (dialog), -1, -1); - - if (gtk_widget_is_drawable (widget)) - { - /* Force a size request of everything before we start. This will make sure - * that widget->requisition is meaningful. - */ - gtk_widget_get_preferred_size (GTK_WIDGET (dialog), &req, NULL); - gtk_widget_get_preferred_size (widget, &widget_req, NULL); - } - - _gtk_file_chooser_embed_get_default_size (GTK_FILE_CHOOSER_EMBED (priv->widget), - &default_width, &default_height); - - gtk_window_resize (GTK_WINDOW (dialog), default_width, default_height); -} - -static void file_chooser_widget_selection_changed (GtkWidget *widget, GtkFileChooserDialog *dialog) { @@ -563,6 +536,23 @@ ensure_default_response (GtkFileChooserDialog *dialog) } static void +gtk_file_chooser_dialog_realize (GtkWidget *widget) +{ + GtkFileChooserDialog *dialog = GTK_FILE_CHOOSER_DIALOG (widget); + GSettings *settings; + gint width, height; + + settings = _gtk_file_chooser_get_settings_for_widget (widget); + g_settings_get (settings, SETTINGS_KEY_WINDOW_SIZE, "(ii)", &width, &height); + + if (width != 0 && height != 0) + gtk_window_set_default_size (GTK_WINDOW (dialog), width, height); + g_print ("%d %d\n", width, height); + + GTK_WIDGET_CLASS (gtk_file_chooser_dialog_parent_class)->realize (widget); +} + +static void gtk_file_chooser_dialog_map (GtkWidget *widget) { GtkFileChooserDialog *dialog = GTK_FILE_CHOOSER_DIALOG (widget); diff --git a/gtk/gtkfilechooserembed.c b/gtk/gtkfilechooserembed.c index 7c64f37961..24c171a730 100644 --- a/gtk/gtkfilechooserembed.c +++ b/gtk/gtkfilechooserembed.c @@ -22,13 +22,8 @@ #include "gtkintl.h" static void gtk_file_chooser_embed_class_init (gpointer g_iface); -static void delegate_get_default_size (GtkFileChooserEmbed *chooser_embed, - gint *default_width, - gint *default_height); static gboolean delegate_should_respond (GtkFileChooserEmbed *chooser_embed); static void delegate_initial_focus (GtkFileChooserEmbed *chooser_embed); -static void delegate_default_size_changed (GtkFileChooserEmbed *chooser_embed, - gpointer data); static void delegate_response_requested (GtkFileChooserEmbed *chooser_embed, gpointer data); @@ -50,7 +45,6 @@ get_delegate (GtkFileChooserEmbed *receiver) void _gtk_file_chooser_embed_delegate_iface_init (GtkFileChooserEmbedIface *iface) { - iface->get_default_size = delegate_get_default_size; iface->should_respond = delegate_should_respond; iface->initial_focus = delegate_initial_focus; } @@ -74,22 +68,12 @@ _gtk_file_chooser_embed_set_delegate (GtkFileChooserEmbed *receiver, g_object_set_data (G_OBJECT (receiver), I_("gtk-file-chooser-embed-delegate"), delegate); - g_signal_connect (delegate, "default-size-changed", - G_CALLBACK (delegate_default_size_changed), receiver); g_signal_connect (delegate, "response-requested", G_CALLBACK (delegate_response_requested), receiver); } -static void -delegate_get_default_size (GtkFileChooserEmbed *chooser_embed, - gint *default_width, - gint *default_height) -{ - _gtk_file_chooser_embed_get_default_size (get_delegate (chooser_embed), default_width, default_height); -} - static gboolean delegate_should_respond (GtkFileChooserEmbed *chooser_embed) { @@ -103,13 +87,6 @@ delegate_initial_focus (GtkFileChooserEmbed *chooser_embed) } static void -delegate_default_size_changed (GtkFileChooserEmbed *chooser_embed, - gpointer data) -{ - g_signal_emit_by_name (data, "default-size-changed"); -} - -static void delegate_response_requested (GtkFileChooserEmbed *chooser_embed, gpointer data) { @@ -149,13 +126,6 @@ gtk_file_chooser_embed_class_init (gpointer g_iface) { GType iface_type = G_TYPE_FROM_INTERFACE (g_iface); - g_signal_new (I_("default-size-changed"), - iface_type, - G_SIGNAL_RUN_LAST, - G_STRUCT_OFFSET (GtkFileChooserEmbedIface, default_size_changed), - NULL, NULL, - NULL, - G_TYPE_NONE, 0); g_signal_new (I_("response-requested"), iface_type, G_SIGNAL_RUN_LAST, @@ -165,18 +135,6 @@ gtk_file_chooser_embed_class_init (gpointer g_iface) G_TYPE_NONE, 0); } -void -_gtk_file_chooser_embed_get_default_size (GtkFileChooserEmbed *chooser_embed, - gint *default_width, - gint *default_height) -{ - g_return_if_fail (GTK_IS_FILE_CHOOSER_EMBED (chooser_embed)); - g_return_if_fail (default_width != NULL); - g_return_if_fail (default_height != NULL); - - GTK_FILE_CHOOSER_EMBED_GET_IFACE (chooser_embed)->get_default_size (chooser_embed, default_width, default_height); -} - gboolean _gtk_file_chooser_embed_should_respond (GtkFileChooserEmbed *chooser_embed) { diff --git a/gtk/gtkfilechooserembed.h b/gtk/gtkfilechooserembed.h index 1e333588fe..5a013190a3 100644 --- a/gtk/gtkfilechooserembed.h +++ b/gtk/gtkfilechooserembed.h @@ -38,24 +38,16 @@ struct _GtkFileChooserEmbedIface /* Methods */ - void (*get_default_size) (GtkFileChooserEmbed *chooser_embed, - gint *default_width, - gint *default_height); - gboolean (*should_respond) (GtkFileChooserEmbed *chooser_embed); void (*initial_focus) (GtkFileChooserEmbed *chooser_embed); /* Signals */ - void (*default_size_changed) (GtkFileChooserEmbed *chooser_embed); void (*response_requested) (GtkFileChooserEmbed *chooser_embed); }; GType _gtk_file_chooser_embed_get_type (void) G_GNUC_CONST; -void _gtk_file_chooser_embed_get_default_size (GtkFileChooserEmbed *chooser_embed, - gint *default_width, - gint *default_height); gboolean _gtk_file_chooser_embed_should_respond (GtkFileChooserEmbed *chooser_embed); void _gtk_file_chooser_embed_initial_focus (GtkFileChooserEmbed *chooser_embed); diff --git a/gtk/gtkfilechooserwidget.c b/gtk/gtkfilechooserwidget.c index f6d5c1b2e6..12988b498c 100644 --- a/gtk/gtkfilechooserwidget.c +++ b/gtk/gtkfilechooserwidget.c @@ -476,7 +476,6 @@ static void gtk_file_chooser_widget_get_property (GObject *obj GValue *value, GParamSpec *pspec); static void gtk_file_chooser_widget_dispose (GObject *object); -static void gtk_file_chooser_widget_realize (GtkWidget *widget); static void gtk_file_chooser_widget_map (GtkWidget *widget); static void gtk_file_chooser_widget_unmap (GtkWidget *widget); static void gtk_file_chooser_widget_root (GtkWidget *widget); @@ -518,9 +517,6 @@ static gboolean gtk_file_chooser_widget_remove_shortcut_folder (GtkFileCho GError **error); static GSList * gtk_file_chooser_widget_list_shortcut_folders (GtkFileChooser *chooser); -static void gtk_file_chooser_widget_get_default_size (GtkFileChooserEmbed *chooser_embed, - gint *default_width, - gint *default_height); static gboolean gtk_file_chooser_widget_should_respond (GtkFileChooserEmbed *chooser_embed); static void gtk_file_chooser_widget_initial_focus (GtkFileChooserEmbed *chooser_embed); @@ -660,7 +656,6 @@ gtk_file_chooser_widget_iface_init (GtkFileChooserIface *iface) static void gtk_file_chooser_embed_default_iface_init (GtkFileChooserEmbedIface *iface) { - iface->get_default_size = gtk_file_chooser_widget_get_default_size; iface->should_respond = gtk_file_chooser_widget_should_respond; iface->initial_focus = gtk_file_chooser_widget_initial_focus; } @@ -946,14 +941,6 @@ change_folder_and_display_error (GtkFileChooserWidget *impl, } static void -emit_default_size_changed (GtkFileChooserWidget *impl) -{ - profile_msg (" emit default-size-changed start", NULL); - g_signal_emit_by_name (impl, "default-size-changed"); - profile_msg (" emit default-size-changed end", NULL); -} - -static void update_preview_widget_visibility (GtkFileChooserWidget *impl) { GtkFileChooserWidgetPrivate *priv = gtk_file_chooser_widget_get_instance_private (impl); @@ -980,9 +967,6 @@ update_preview_widget_visibility (GtkFileChooserWidget *impl) gtk_widget_show (priv->preview_box); else gtk_widget_hide (priv->preview_box); - - if (!gtk_widget_get_mapped (GTK_WIDGET (impl))) - emit_default_size_changed (impl); } static void @@ -3226,8 +3210,6 @@ update_appearance (GtkFileChooserWidget *impl) * of files may change depending whether we are in a file or folder-only mode. */ gtk_widget_queue_draw (priv->browse_files_tree_view); - - emit_default_size_changed (impl); } static gchar * @@ -3679,8 +3661,6 @@ gtk_file_chooser_widget_style_updated (GtkWidget *widget) change_icon_theme (impl); - emit_default_size_changed (impl); - profile_end ("end", NULL); } @@ -3789,19 +3769,6 @@ settings_save (GtkFileChooserWidget *impl) g_settings_apply (settings); } -/* GtkWidget::realize method */ -static void -gtk_file_chooser_widget_realize (GtkWidget *widget) -{ - GtkFileChooserWidget *impl; - - impl = GTK_FILE_CHOOSER_WIDGET (widget); - - GTK_WIDGET_CLASS (gtk_file_chooser_widget_parent_class)->realize (widget); - - emit_default_size_changed (impl); -} - /* Changes the current folder to $CWD */ static void switch_to_cwd (GtkFileChooserWidget *impl) @@ -6173,64 +6140,6 @@ gtk_file_chooser_widget_list_shortcut_folders (GtkFileChooser *chooser) return gtk_places_sidebar_list_shortcuts (GTK_PLACES_SIDEBAR (priv->places_sidebar)); } -/* Guesses a size based upon font sizes */ -static void -find_good_size_from_style (GtkWidget *widget, - gint *width, - gint *height) -{ - GtkCssStyle *style; - double font_size; - - style = gtk_css_node_get_style (gtk_widget_get_css_node (widget)); - font_size = _gtk_css_number_value_get (gtk_css_style_get_value (style, GTK_CSS_PROPERTY_FONT_SIZE), 100); - - *width = font_size * NUM_CHARS; - *height = font_size * NUM_LINES; -} - -static void -gtk_file_chooser_widget_get_default_size (GtkFileChooserEmbed *chooser_embed, - gint *default_width, - gint *default_height) -{ - GtkFileChooserWidget *impl = GTK_FILE_CHOOSER_WIDGET (chooser_embed); - GtkFileChooserWidgetPrivate *priv = gtk_file_chooser_widget_get_instance_private (impl); - GtkRequisition req; - int width, height; - GSettings *settings; - - settings = _gtk_file_chooser_get_settings_for_widget (GTK_WIDGET (impl)); - - g_settings_get (settings, SETTINGS_KEY_WINDOW_SIZE, "(ii)", &width, &height); - - if (width > 0 && height > 0) - { - *default_width = width; - *default_height = height; - return; - } - - find_good_size_from_style (GTK_WIDGET (chooser_embed), default_width, default_height); - - if (priv->preview_widget_active && - priv->preview_widget && - gtk_widget_get_visible (priv->preview_widget)) - { - gtk_widget_get_preferred_size (priv->preview_box, - &req, NULL); - *default_width += PREVIEW_HBOX_SPACING + req.width; - } - - if (priv->extra_widget && - gtk_widget_get_visible (priv->extra_widget)) - { - gtk_widget_get_preferred_size (priv->extra_align, - &req, NULL); - *default_height += gtk_box_get_spacing (GTK_BOX (priv->box)) + req.height; - } -} - struct switch_folder_closure { GtkFileChooserWidget *impl; GFile *file; @@ -8032,7 +7941,6 @@ gtk_file_chooser_widget_class_init (GtkFileChooserWidgetClass *class) gobject_class->get_property = gtk_file_chooser_widget_get_property; gobject_class->dispose = gtk_file_chooser_widget_dispose; - widget_class->realize = gtk_file_chooser_widget_realize; widget_class->map = gtk_file_chooser_widget_map; widget_class->unmap = gtk_file_chooser_widget_unmap; widget_class->root = gtk_file_chooser_widget_root; diff --git a/gtk/gtkfontchooserwidget.c b/gtk/gtkfontchooserwidget.c index 0ad56e8318..c25a4889b3 100644 --- a/gtk/gtkfontchooserwidget.c +++ b/gtk/gtkfontchooserwidget.c @@ -1111,17 +1111,14 @@ gtk_font_chooser_widget_get_preview_text_height (GtkFontChooserWidget *fontchoos GtkFontChooserWidgetPrivate *priv = gtk_font_chooser_widget_get_instance_private (fontchooser); GtkWidget *treeview = priv->family_face_list; GtkStyleContext *context; - double dpi, font_size; + double font_size; context = gtk_widget_get_style_context (treeview); - dpi = _gtk_css_number_value_get (_gtk_style_context_peek_property (context, - GTK_CSS_PROPERTY_DPI), - 100); - gtk_style_context_get (context, - "font-size", &font_size, - NULL); - - return (dpi < 0.0 ? 96.0 : dpi) / 72.0 * PANGO_SCALE_X_LARGE * font_size * PANGO_SCALE; + font_size = _gtk_css_number_value_get (_gtk_style_context_peek_property (context, + GTK_CSS_PROPERTY_FONT_SIZE), + 100); + + return PANGO_SCALE_X_LARGE * font_size * PANGO_SCALE; } static PangoAttrList * diff --git a/gtk/ui/gtkfilechooserdialog.ui b/gtk/ui/gtkfilechooserdialog.ui index 4434b186de..425dab20e6 100644 --- a/gtk/ui/gtkfilechooserdialog.ui +++ b/gtk/ui/gtkfilechooserdialog.ui @@ -17,7 +17,6 @@ <child> <object class="GtkFileChooserWidget" id="widget"> <property name="expand">1</property> - <signal name="default-size-changed" handler="file_chooser_widget_default_size_changed" swapped="no"/> <signal name="file-activated" handler="file_chooser_widget_file_activated" swapped="no"/> <signal name="response-requested" handler="file_chooser_widget_response_requested" swapped="no"/> <signal name="selection-changed" handler="file_chooser_widget_selection_changed" swapped="no"/> |