aboutsummaryrefslogtreecommitdiffstats
path: root/ui/gtk/gui_utils.c
diff options
context:
space:
mode:
authorBill Meier <wmeier@newsguy.com>2013-01-10 20:46:58 +0000
committerBill Meier <wmeier@newsguy.com>2013-01-10 20:46:58 +0000
commit9e1bf0acefff3efe9c67c40f66b562f8fdbb837e (patch)
treeb77862a21b5526f6d74a94aa774d055a55960b44 /ui/gtk/gui_utils.c
parent708de14dc86e6db652023ab21323e53c6cb429f0 (diff)
Update comments.
svn path=/trunk/; revision=47021
Diffstat (limited to 'ui/gtk/gui_utils.c')
-rw-r--r--ui/gtk/gui_utils.c12
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);