diff options
author | Bill Meier <wmeier@newsguy.com> | 2013-01-10 20:46:58 +0000 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2013-01-10 20:46:58 +0000 |
commit | 9e1bf0acefff3efe9c67c40f66b562f8fdbb837e (patch) | |
tree | b77862a21b5526f6d74a94aa774d055a55960b44 /ui/gtk/gui_utils.c | |
parent | 708de14dc86e6db652023ab21323e53c6cb429f0 (diff) |
Update comments.
svn path=/trunk/; revision=47021
Diffstat (limited to 'ui/gtk/gui_utils.c')
-rw-r--r-- | ui/gtk/gui_utils.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/ui/gtk/gui_utils.c b/ui/gtk/gui_utils.c index 2c7c72aee2..f3c5a3ddb6 100644 --- a/ui/gtk/gui_utils.c +++ b/ui/gtk/gui_utils.c @@ -2064,16 +2064,20 @@ ws_gtk_grid_attach_extended(GtkGrid *grid, GtkWidget *child, * Default for EXPAND is "not set". * In this case "computed expand" based on any child(ren) of this widget will * affect this widget. - * If EXPAND is set (either TRUE or FALSE) then the value overrides any effect from children. - * - * This wrapper currently only sets EXPAND (to TRUE) if the GTK_FILL is specified; + * If EXPAND is set (either TRUE or FALSE) then the value overrides any effect + * from children. */ + + /* Note: widget defaults are FALSE */ if (xoptions & GTK_EXPAND) gtk_widget_set_hexpand(child, TRUE); if (yoptions & GTK_EXPAND) gtk_widget_set_vexpand(child, TRUE); - /* Note: default is GTK_FILL */ + /* Note: widget default is GTK_FILL */ + /* XXX: Is an 'align' ignored if the corresponding 'fill; is FALSE ? */ + /* XXX: don't set FILL(since is dedault) but just clear if not set ?? */ + /* ToDo: review effect of explicit set/clear vs explict clear only */ gtk_widget_set_halign(child, (xoptions & GTK_FILL) ? GTK_ALIGN_FILL : GTK_ALIGN_CENTER); gtk_widget_set_valign(child, (yoptions & GTK_FILL) ? GTK_ALIGN_FILL : GTK_ALIGN_CENTER); |