nhyatt/putty-source
1
0
Fork 0
mirror of https://git.tartarus.org/simon/putty.git synced 2025-01-10 01:48:00 +00:00
Code Issues Projects Releases Wiki Activity

Fix GTK3 size calculations in unifontsel_new.

Browse Source 
It turns out that in GTK3, if you instantiate a GtkLabel and
immediately try to find out its preferred size, you get back zero for
both dimensions. Presumably none of that gets figured out properly
until the widget is displayed, or some such.

However, you can retrieve the PangoLayout from the label immediately
and ask Pango for the dimensions of that. That seems like a bit of a
bodge, but it works! The GTK3 unifont selector now comes out with all
the interface elements in sensible sizes - in particular, the preview
drawing area now has non-zero height.
This commit is contained in:
Simon Tatham 2015-08-22 11:46:05 +01:00
parent 37ec0b463f
commit 03e21b7cad
1 changed files with 46 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

62
unix/gtkfont.c
View File

@ -2914,6 +2914,39 @@ static gint unifontsel_configure_area(GtkWidget *widget,
return TRUE; return TRUE;
} }
static void get_label_text_dimensions(const char *text,
int *width, int *height)
{
/*
* Determine the dimensions of a piece of text in the standard
* font used in GTK interface elements like labels. We do this by
* instantiating an actual GtkLabel, and then querying its size.
*
* But GTK2 and GTK3 require us to query the size completely
* differently. I'm sure there ought to be an easier approach than
* the way I'm doing this in GTK3, too!
*/
GtkWidget *label = gtk_label_new(text);
#if GTK_CHECK_VERSION(3,0,0)
PangoLayout *layout = gtk_label_get_layout(GTK_LABEL(label));
PangoRectangle logrect;
pango_layout_get_extents(layout, NULL, &logrect);
*width = logrect.width / PANGO_SCALE;
*height = logrect.height / PANGO_SCALE;
#else
GtkRequisition req;
gtk_widget_size_request(label, &req);
*width = req.width;
*height = req.height;
#endif
g_object_ref_sink(G_OBJECT(label));
#if GTK_CHECK_VERSION(2,10,0)
g_object_unref(label);
#endif
}
unifontsel *unifontsel_new(const char *wintitle) unifontsel *unifontsel_new(const char *wintitle)
{ {
unifontsel_internal *fs = snew(unifontsel_internal); unifontsel_internal *fs = snew(unifontsel_internal);
@ -2927,25 +2960,22 @@ unifontsel *unifontsel_new(const char *wintitle)
fs->selected = NULL; fs->selected = NULL;
{ {
int width, height;
/* /*
* Invent some magic size constants. * Invent some magic size constants.
*/ */
GtkRequisition req; get_label_text_dimensions("Quite Long Font Name (Foundry)",
label = gtk_label_new("Quite Long Font Name (Foundry)"); &width, &height);
gtk_widget_size_request(label, &req); font_width = width;
font_width = req.width; lists_height = 14 * height;
lists_height = 14 * req.height; preview_height = 5 * height;
preview_height = 5 * req.height;
gtk_label_set_text(GTK_LABEL(label), "Italic Extra Condensed"); get_label_text_dimensions("Italic Extra Condensed", &width, &height);
gtk_widget_size_request(label, &req); style_width = width;
style_width = req.width;
gtk_label_set_text(GTK_LABEL(label), "48000"); get_label_text_dimensions("48000", &width, &height);
gtk_widget_size_request(label, &req); size_width = width;
size_width = req.width;
g_object_ref_sink(G_OBJECT(label));
#if GTK_CHECK_VERSION(2,10,0)
g_object_unref(label);
#endif
} }
/* /*