11 files changed, 255 insertions, 83 deletions

diff --git a/src/ui/bindingswidget.c b/src/ui/bindingswidget.c
index c73982b3..dc23dd6e 100644
--- a/src/ui/bindingswidget.c
+++ b/src/ui/bindingswidget.c
@@ -110,6 +110,7 @@ static void updateItems_BindingsWidget_(iBindingsWidget *d) {
 void init_BindingsWidget(iBindingsWidget *d) {
     iWidget *w = as_Widget(d);
     init_Widget(w);
+    setId_Widget(w, "bindings");
     setFlags_Widget(w, resizeChildren_WidgetFlag, iTrue);
     d->activePos = iInvalidPos;
     d->contextPos = iInvalidPos;
diff --git a/src/ui/certimportwidget.c b/src/ui/certimportwidget.c
index 43e8ff97..a4245603 100644
--- a/src/ui/certimportwidget.c
+++ b/src/ui/certimportwidget.c
@@ -148,8 +148,8 @@ void init_CertImportWidget(iCertImportWidget *d) {
     }
     addChild_Widget(w, iClob(page));
     arrange_Widget(w);
-    setSize_Widget(as_Widget(d->crtLabel), init_I2(width_Widget(w) - 6.5 * gap_UI, gap_UI * 12));
-    setSize_Widget(as_Widget(d->keyLabel), init_I2(width_Widget(w) - 6.5 * gap_UI, gap_UI * 12));
+    setFixedSize_Widget(as_Widget(d->crtLabel), init_I2(width_Widget(w) - 6.5 * gap_UI, gap_UI * 12));
+    setFixedSize_Widget(as_Widget(d->keyLabel), init_I2(width_Widget(w) - 6.5 * gap_UI, gap_UI * 12));
     /* Buttons. */
     addChild_Widget(w, iClob(makePadding_Widget(gap_UI)));
     iWidget *buttons = makeDialogButtons_Widget(
diff --git a/src/ui/documentwidget.c b/src/ui/documentwidget.c
index 07b359da..9a50b611 100644
--- a/src/ui/documentwidget.c
+++ b/src/ui/documentwidget.c
@@ -1752,7 +1752,7 @@ static iBool handleCommand_DocumentWidget_(iDocumentWidget *d, const char *cmd)
         delete_Array(items);
         /* Enforce a minimum size. */
         iWidget *sizer = new_Widget();
-        setSize_Widget(sizer, init_I2(gap_UI * 65, 1));
+        setFixedSize_Widget(sizer, init_I2(gap_UI * 65, 1));
         addChildFlags_Widget(dlg, iClob(sizer), frameless_WidgetFlag);
         setFlags_Widget(dlg, centerHorizontal_WidgetFlag, iFalse);
         if (deviceType_App() != phone_AppDeviceType) {
diff --git a/src/ui/inputwidget.c b/src/ui/inputwidget.c
index 5593b5b9..0856db93 100644
--- a/src/ui/inputwidget.c
+++ b/src/ui/inputwidget.c
@@ -248,7 +248,7 @@ void setMaxLen_InputWidget(iInputWidget *d, size_t maxLen) {
         /* Set a fixed size. */
         iBlock *content = new_Block(maxLen);
         fill_Block(content, 'M');
-        setSize_Widget(
+        setFixedSize_Widget(
             as_Widget(d),
             add_I2(measure_Text(d->font, cstr_Block(content)), init_I2(6 * gap_UI, 2 * gap_UI)));
         delete_Block(content);
diff --git a/src/ui/lookupwidget.c b/src/ui/lookupwidget.c
index c711d61f..f3fd54d2 100644
--- a/src/ui/lookupwidget.c
+++ b/src/ui/lookupwidget.c
@@ -657,8 +657,8 @@ static iBool processEvent_LookupWidget_(iLookupWidget *d, const SDL_Event *ev) {
             const iWindow *window = get_Window();
             const iInt2 rootSize = rootSize_Window(window);
             const iRect navBarBounds = bounds_Widget(findWidget_App("navbar"));
-            setSize_Widget(w, init_I2(width_Widget(findWidget_App("url")),
-                                      (rootSize.y - bottom_Rect(navBarBounds)) / 2));
+            setFixedSize_Widget(w, init_I2(width_Widget(findWidget_App("url")),
+                                           (rootSize.y - bottom_Rect(navBarBounds)) / 2));
             setPos_Widget(w, bottomLeft_Rect(bounds_Widget(findWidget_App("url"))));
 #if defined (iPlatformAppleMobile)
             /* Adjust height based on keyboard size. */ {
diff --git a/src/ui/translation.c b/src/ui/translation.c
index 86607fc6..4102fcb9 100644
--- a/src/ui/translation.c
+++ b/src/ui/translation.c
@@ -442,8 +442,9 @@ iBool handleCommand_Translation(iTranslation *d, const char *cmd) {
             setFlags_Widget(as_Widget(acceptButton), disabled_WidgetFlag, iTrue);
             iTranslationProgressWidget *prog = new_TranslationProgressWidget();
             setPos_Widget(as_Widget(prog), langs->rect.pos);
-            setSize_Widget(as_Widget(prog), langs->rect.size);
-            addChild_Widget(d->dlg, iClob(prog));
+            setFixedSize_Widget(as_Widget(prog), init_I2(width_Rect(innerBounds_Widget(d->dlg)),
+                                                         langs->rect.size.y));
+            addChildFlags_Widget(d->dlg, iClob(prog), 0);
             submit_Translation(d);
         }
         return iTrue;
diff --git a/src/ui/util.c b/src/ui/util.c
index 41b8f65c..564cf785 100644
--- a/src/ui/util.c
+++ b/src/ui/util.c
@@ -438,7 +438,7 @@ iInt2 delta_Click(const iClick *d) {
 iWidget *makePadding_Widget(int size) {
     iWidget *pad = new_Widget();
     setId_Widget(pad, "padding");
-    setSize_Widget(pad, init1_I2(size));
+    setFixedSize_Widget(pad, init1_I2(size));
     return pad;
 }
 
@@ -463,7 +463,7 @@ iWidget *makeHDiv_Widget(void) {
 
 iWidget *addAction_Widget(iWidget *parent, int key, int kmods, const char *command) {
     iLabelWidget *action = newKeyMods_LabelWidget("", key, kmods, command);
-    setSize_Widget(as_Widget(action), zero_I2());
+    setFixedSize_Widget(as_Widget(action), zero_I2());
     addChildFlags_Widget(parent, iClob(action), hidden_WidgetFlag);
     return as_Widget(action);
 }
@@ -856,16 +856,23 @@ iWidget *removeTabPage_Widget(iWidget *tabs, size_t index) {
 }
 
 void resizeToLargestPage_Widget(iWidget *tabs) {
+//    puts("RESIZE TO LARGEST PAGE ...");
+    iWidget *pages = findChild_Widget(tabs, "tabs.pages");
+    iForEach(ObjectList, i, children_Widget(pages)) {
+        setMinSize_Widget(i.object, zero_I2());
+//        resetSize_Widget(i.object);
+    }
     arrange_Widget(tabs);
     iInt2 largest = zero_I2();
-    iWidget *pages = findChild_Widget(tabs, "tabs.pages");
-    iConstForEach(ObjectList, i, children_Widget(pages)) {
-        largest = max_I2(largest, ((const iWidget *) i.object)->rect.size);
+    iConstForEach(ObjectList, j, children_Widget(pages)) {
+        const iWidget *page = constAs_Widget(j.object);
+        largest = max_I2(largest, page->rect.size);
     }
-    iForEach(ObjectList, j, children_Widget(pages)) {
-        setSize_Widget(j.object, largest);
+    iForEach(ObjectList, k, children_Widget(pages)) {
+        setMinSize_Widget(k.object, largest);
     }
-    setSize_Widget(tabs, addY_I2(largest, height_Widget(findChild_Widget(tabs, "tabs.buttons"))));
+    setFixedSize_Widget(tabs, addY_I2(largest, height_Widget(findChild_Widget(tabs, "tabs.buttons"))));
+//    puts("... DONE WITH RESIZE TO LARGEST PAGE");
 }
 
 iLabelWidget *tabButtonForPage_Widget_(iWidget *tabs, const iWidget *page) {
@@ -1012,7 +1019,7 @@ static void updateSheetPanelMetrics_(iWidget *sheet) {
         setSize_Widget(pad, init1_I2(naviHeight));
     }
 #endif
-    setSize_Widget(navi, init_I2(-1, naviHeight));
+    setFixedSize_Widget(navi, init_I2(-1, naviHeight));
 }
 
 static iBool slidePanelHandler_(iWidget *d, const char *cmd) {
@@ -1786,12 +1793,16 @@ static iBool toggleHandler_(iWidget *d, const char *cmd) {
                                        isSelected_Widget(d) ? 1 : 0));
         return iTrue;
     }
+    else if (equal_Command(cmd, "lang.changed")) {
+        /* TODO: Measure labels again. */
+    }
     return iFalse;
 }
 
 iWidget *makeToggle_Widget(const char *id) {
     iWidget *toggle = as_Widget(new_LabelWidget("${toggle.yes}", "toggle")); /* "YES" for sizing */
     setId_Widget(toggle, id);
+    /* TODO: Measure both labels and use the larger of the two. */
     updateTextCStr_LabelWidget((iLabelWidget *) toggle, "${toggle.no}"); /* actual initial value */
     setFlags_Widget(toggle, fixedWidth_WidgetFlag, iTrue);
     setCommandHandler_Widget(toggle, toggleHandler_);
@@ -1879,6 +1890,36 @@ static int cmp_MenuItem_(const void *e1, const void *e2) {
 }
 #endif
 
+void updatePreferencesLayout_Widget(iWidget *prefs) {
+    if (!prefs) return;
+    /* Doing manual layout here because the widget arranging logic isn't sophisticated enough. */
+    /* TODO: Make the arranging more sophisticated to automate this. */
+    static const char *inputIds[] = {
+        "prefs.searchurl",
+        "prefs.downloads",
+        "prefs.ca.file",
+        "prefs.ca.path",
+        "prefs.proxy.gemini",
+        "prefs.proxy.gopher",
+        "prefs.proxy.http"
+    };
+    iWidget *tabs = findChild_Widget(prefs, "prefs.tabs");
+    /* Input fields expand to the right edge. */
+    /* TODO: Add an arrangement flag for this. */
+    iForIndices(i, inputIds) {
+        iInputWidget *input = findChild_Widget(tabs, inputIds[i]);
+        if (input) {
+            as_Widget(input)->rect.size.x = 0;
+        }
+    }
+    as_Widget(findChild_Widget(prefs, "bindings"))->rect.size.x = 0;
+    resizeToLargestPage_Widget(tabs);
+    arrange_Widget(prefs);
+    iForIndices(i, inputIds) {
+        expandInputFieldWidth_(findChild_Widget(tabs, inputIds[i]));
+    }
+}
+
 iWidget *makePreferences_Widget(void) {
     iWidget *dlg = makeSheet_Widget("prefs");
     addChildFlags_Widget(dlg,
@@ -2021,7 +2062,7 @@ iWidget *makePreferences_Widget(void) {
         addChildFlags_Widget(values, iClob(sats), arrangeHorizontal_WidgetFlag | arrangeSize_WidgetFlag);
     }
     /* Layout. */ {
-        appendTwoColumnPage_(tabs, "${heading.prefs.style}", '4', &headings, &values);
+        setId_Widget(appendTwoColumnPage_(tabs, "${heading.prefs.style}", '4', &headings, &values), "prefs.page.style");
         makeTwoColumnHeading_("${heading.prefs.fonts}", headings, values);
         /* Fonts. */ {
             addChild_Widget(headings, iClob(makeHeading_Widget("${prefs.headingfont}")));
@@ -2096,22 +2137,13 @@ iWidget *makePreferences_Widget(void) {
         appendFramelessTabPage_(tabs, iClob(bind), "${heading.prefs.keys}", '6', KMOD_PRIMARY);
     }
     addChild_Widget(dlg, iClob(makePadding_Widget(gap_UI)));
-    resizeToLargestPage_Widget(tabs);
-    arrange_Widget(dlg);
-    /* Set input field sizes. */ {
-        expandInputFieldWidth_(findChild_Widget(tabs, "prefs.searchurl"));
-        expandInputFieldWidth_(findChild_Widget(tabs, "prefs.downloads"));
-        expandInputFieldWidth_(findChild_Widget(tabs, "prefs.ca.file"));
-        expandInputFieldWidth_(findChild_Widget(tabs, "prefs.ca.path"));
-        expandInputFieldWidth_(findChild_Widget(tabs, "prefs.proxy.gemini"));
-        expandInputFieldWidth_(findChild_Widget(tabs, "prefs.proxy.gopher"));
-        expandInputFieldWidth_(findChild_Widget(tabs, "prefs.proxy.http"));
-    }
+    updatePreferencesLayout_Widget(dlg);
     addChild_Widget(dlg,
                     iClob(makeDialogButtons_Widget(
                         (iMenuItem[]){ { "${dismiss}", SDLK_ESCAPE, 0, "prefs.dismiss" } }, 1)));
     addChild_Widget(get_Window()->root, iClob(dlg));
     finalizeSheet_Widget(dlg);
+    //printTree_Widget(dlg);
     return dlg;
 }
 
diff --git a/src/ui/util.h b/src/ui/util.h
index c6320e29..02b0df38 100644
--- a/src/ui/util.h
+++ b/src/ui/util.h
@@ -228,7 +228,9 @@ iWidget *   makeMessage_Widget      (const char *title, const char *msg);
 iWidget *   makeQuestion_Widget     (const char *title, const char *msg,
                                      const iMenuItem *items, size_t numItems);
 
-iWidget *   makePreferences_Widget      (void);
+iWidget *   makePreferences_Widget          (void);
+void        updatePreferencesLayout_Widget  (iWidget *prefs);
+
 iWidget *   makeBookmarkEditor_Widget   (void);
 iWidget *   makeBookmarkCreation_Widget (const iString *url, const iString *title, iChar icon);
 iWidget *   makeIdentityCreation_Widget (void);
diff --git a/src/ui/widget.c b/src/ui/widget.c
index 91f6c12d..baabf673 100644
--- a/src/ui/widget.c
+++ b/src/ui/widget.c
@@ -86,6 +86,7 @@ void init_Widget(iWidget *d) {
     init_String(&d->id);
     d->flags          = 0;
     d->rect           = zero_Rect();
+    d->minSize        = zero_I2();
     d->bgColor        = none_ColorId;
     d->frameColor     = none_ColorId;
     init_Anim(&d->visualOffset, 0.0f);
@@ -188,20 +189,25 @@ void setPos_Widget(iWidget *d, iInt2 pos) {
     setFlags_Widget(d, fixedPosition_WidgetFlag, iTrue);
 }
 
-void setSize_Widget(iWidget *d, iInt2 size) {
+void setFixedSize_Widget(iWidget *d, iInt2 fixedSize) {
     int flags = fixedSize_WidgetFlag;
-    if (size.x < 0) {
-        size.x = d->rect.size.x;
+    if (fixedSize.x < 0) {
+        fixedSize.x = d->rect.size.x;
         flags &= ~fixedWidth_WidgetFlag;
     }
-    if (size.y < 0) {
-        size.y = d->rect.size.y;
+    if (fixedSize.y < 0) {
+        fixedSize.y = d->rect.size.y;
         flags &= ~fixedHeight_WidgetFlag;
     }
-    d->rect.size = size;
+    d->rect.size = fixedSize;
     setFlags_Widget(d, flags, iTrue);
 }
 
+void setMinSize_Widget(iWidget *d, iInt2 minSize) {
+    d->minSize = minSize;
+    /* rearranging needed to apply this */
+}
+
 void setPadding_Widget(iWidget *d, int left, int top, int right, int bottom) {
     d->padding[0] = left;
     d->padding[1] = top;
@@ -268,33 +274,79 @@ static int widestChild_Widget_(const iWidget *d) {
     return width;
 }
 
+static void arrange_Widget_(iWidget *);
+static const iBool tracing_ = iFalse;
+
+#define TRACE(d, ...)   if (tracing_) { printf_Widget_(d, __VA_ARGS__); }
+
+static int depth_Widget_(const iWidget *d) {
+    int depth = 0;
+    for (const iWidget *w = d->parent; w; w = w->parent) {
+        depth++;
+    }
+    return depth;
+}
+
+static void printf_Widget_(const iWidget *d, const char *format, ...) {
+    va_list args;
+    va_start(args, format);
+    iString *msg = new_String();
+    for (size_t i = 0; i < depth_Widget_(d); ++i) {
+        appendCStr_String(msg, "|   ");
+    }
+    appendFormat_String(msg, "[%p] %s(%s) ", d, class_Widget(d)->name, cstr_String(id_Widget(d)));
+    while (size_String(msg) < 44 + depth_Widget_(d) * 4) {
+        appendCStr_String(msg, " ");
+    }
+    iBlock *msg2 = new_Block(0);
+    vprintf_Block(msg2, format, args);
+    va_end(args);
+    printf("%s%s\n", cstr_String(msg), cstr_Block(msg2));
+    delete_Block(msg2);
+    delete_String(msg);
+}
+
 static void setWidth_Widget_(iWidget *d, int width) {
     iAssert(width >= 0);
+    TRACE(d, "attempt to set width to %d (current: %d, min width: %d)", width, d->rect.size.x, d->minSize.x);
+    width = iMax(width, d->minSize.x);
     if (~d->flags & fixedWidth_WidgetFlag || d->flags & collapse_WidgetFlag) {
         if (d->rect.size.x != width) {
             d->rect.size.x = width;
+            TRACE(d, "width has changed to %d", width);
             if (class_Widget(d)->sizeChanged) {
                 const int oldHeight = d->rect.size.y;
                 class_Widget(d)->sizeChanged(d);
                 if (d->rect.size.y != oldHeight) {
+                    TRACE(d, "sizeChanged() cuased height change to %d; redoing parent", d->rect.size.y);
                     /* Widget updated its height. */
-                    arrange_Widget(d->parent);
+                    arrange_Widget_(d->parent);
+                    TRACE(d, "parent layout redone");
                 }
             }
         }
     }
+    else {
+        TRACE(d, "changing width not allowed; flags: %x", d->flags);
+    }
 }
 
 static void setHeight_Widget_(iWidget *d, int height) {
     iAssert(height >= 0);
+    TRACE(d, "attempt to set height to %d (current: %d, min height: %d)", height, d->rect.size.y, d->minSize.y);
+    height = iMax(height, d->minSize.y);
     if (~d->flags & fixedHeight_WidgetFlag || d->flags & collapse_WidgetFlag) {
         if (d->rect.size.y != height) {
             d->rect.size.y = height;
+            TRACE(d, "height has changed to %d", height);
             if (class_Widget(d)->sizeChanged) {
                 class_Widget(d)->sizeChanged(d);
             }
         }
     }
+    else {
+        TRACE(d, "changing height not allowed; flags: %x", d->flags);
+    }
 }
 
 iLocalDef iBool isCollapsed_Widget_(const iWidget *d) {
@@ -336,29 +388,63 @@ static void centerHorizontal_Widget_(iWidget *d) {
                                 : rootSize_Window(get_Window()).x) -
                      width_Rect(d->rect)) /
                     2;
+    TRACE(d, "center horizontally: %d", d->rect.pos.x);
 }
 
-void arrange_Widget(iWidget *d) {
+static void boundsOfChildren_Widget_(const iWidget *d, iRect *bounds_out) {
+    *bounds_out = zero_Rect();
+    iConstForEach(ObjectList, i, d->children) {
+        const iWidget *child = constAs_Widget(i.object);
+        if (isCollapsed_Widget_(child)) {
+            continue;
+        }
+        iRect childRect = child->rect;
+        if (child->flags & ignoreForParentWidth_WidgetFlag) {
+            childRect.size.x = 0;
+        }
+        if (isEmpty_Rect(*bounds_out)) {
+            *bounds_out = childRect;
+        }
+        else {
+            *bounds_out = union_Rect(*bounds_out, childRect);
+        }
+    }
+}
+
+static void arrange_Widget_(iWidget *d) {
+    TRACE(d, "arranging...");
     if (isCollapsed_Widget_(d)) {
+        TRACE(d, "collapsed => END");
         setFlags_Widget(d, wasCollapsed_WidgetFlag, iTrue);
         return;
     }
     if (d->flags & moveToParentLeftEdge_WidgetFlag) {
         d->rect.pos.x = d->padding[0]; /* FIXME: Shouldn't this be d->parent->padding[0]? */
+        TRACE(d, "move to parent left edge: %d", d->rect.pos.x);
     }
     else if (d->flags & moveToParentRightEdge_WidgetFlag) {
         d->rect.pos.x = width_Rect(innerRect_Widget_(d->parent)) - width_Rect(d->rect);
+        TRACE(d, "move to parent right edge: %d", d->rect.pos.x);
     }
     else if (d->flags & moveToParentBottomEdge_WidgetFlag) {
         d->rect.pos.y = height_Rect(innerRect_Widget_(d->parent)) - height_Rect(d->rect);
+        TRACE(d, "move to parent bottom edge: %d", d->rect.pos.y);
     }
     else if (d->flags & centerHorizontal_WidgetFlag) {
         centerHorizontal_Widget_(d);
     }
     if (d->flags & resizeToParentWidth_WidgetFlag) {
-        setWidth_Widget_(d, width_Rect(innerRect_Widget_(d->parent)));
+        iRect childBounds = zero_Rect();
+        if (flags_Widget(d->parent) & arrangeWidth_WidgetFlag) {
+            /* Can't go narrower than what the children require, though. */
+            boundsOfChildren_Widget_(d, &childBounds);
+        }
+        TRACE(d, "resize to parent width; child bounds width %d", childBounds.size.x, childBounds.size.y);
+        setWidth_Widget_(d, iMaxi(width_Rect(innerRect_Widget_(d->parent)),
+                                  width_Rect(childBounds)));
     }
     if (d->flags & resizeToParentHeight_WidgetFlag) {
+        TRACE(d, "resize to parent height");
         setHeight_Widget_(d, height_Rect(innerRect_Widget_(d->parent)));
     }
     if (d->flags & safePadding_WidgetFlag) {
@@ -370,46 +456,44 @@ void arrange_Widget(iWidget *d) {
     }
     /* The rest of the arrangement depends on child widgets. */
     if (!d->children) {
+        TRACE(d, "no children => END");
         return;
     }
     const size_t childCount = numArrangedChildren_Widget_(d);
-    /* There may still be unarranged children that need arranging internally. */
-    if (childCount == 0) {
-        iForEach(ObjectList, i, d->children) {
-            iWidget *child = as_Widget(i.object);
-            if (isArranged_Widget_(child)) {
-                arrange_Widget(child);
-            }
-        }
-        return;
-    }
+    TRACE(d, "%d arranged children", childCount);
     /* Resize children to fill the parent widget. */
     if (d->flags & resizeChildren_WidgetFlag) {
         const iInt2 dirs = init_I2((d->flags & resizeWidthOfChildren_WidgetFlag) != 0,
                                    (d->flags & resizeHeightOfChildren_WidgetFlag) != 0);
+        TRACE(d, "resize children, x:%d y:%d", dirs.x, dirs.y);
         /* Collapse hidden children. */
         iBool collapseChanged = iFalse;
         iForEach(ObjectList, c, d->children) {
             iWidget *child = as_Widget(c.object);
             if (!isCollapsed_Widget_(child) && child->flags & wasCollapsed_WidgetFlag) {
                 setFlags_Widget(child, wasCollapsed_WidgetFlag, iFalse);
+                TRACE(d, "child %p is uncollapsed", child);
                 /* Undo collapse and determine the normal size again. */
-                arrange_Widget(d);
+                arrange_Widget_(d);
                 collapseChanged = iTrue;
             }
             else if (isCollapsed_Widget_(child) && ~child->flags & wasCollapsed_WidgetFlag) {
                 setFlags_Widget(child, wasCollapsed_WidgetFlag, iTrue);
                 collapseChanged = iTrue;
+                TRACE(d, "child %p flagged as collapsed", child);
             }
         }
         if (collapseChanged) {
-            arrange_Widget(d); /* Redo with the new child sizes. */
+            TRACE(d, "redoing arrangement due to changes in child collapse state");
+            arrange_Widget_(d); /* Redo with the new child sizes. */
             return;
         }
         const int expCount = numExpandingChildren_Widget_(d);
+        TRACE(d, "%d expanding children", expCount);
         /* Only resize the expanding children, not touching the others. */
         if (expCount > 0) {
             iInt2 avail = innerRect_Widget_(d).size;
+            TRACE(d, "inner size: %dx%d", avail.x, avail.y);
             iConstForEach(ObjectList, i, d->children) {
                 const iWidget *child = constAs_Widget(i.object);
                 if (!isArranged_Widget_(child)) {
@@ -420,9 +504,11 @@ void arrange_Widget(iWidget *d) {
                 }
             }
             avail = divi_I2(max_I2(zero_I2(), avail), expCount);
+            TRACE(d, "changing child sizes (expand mode)...");
             iForEach(ObjectList, j, d->children) {
                 iWidget *child = as_Widget(j.object);
                 if (!isArranged_Widget_(child)) {
+                    TRACE(d, "child %p is not arranged", child);
                     continue;
                 }
                 if (child->flags & expand_WidgetFlag) {
@@ -445,6 +531,7 @@ void arrange_Widget(iWidget *d) {
                     }
                 }
             }
+            TRACE(d, "...done changing child sizes (expand mode)");
         }
         else {
             /* Evenly size all children. */
@@ -458,6 +545,7 @@ void arrange_Widget(iWidget *d) {
                 childSize.y /= childCount;
                 unpaddedChildSize.y /= childCount;
             }
+            TRACE(d, "begin changing child sizes (EVEN mode)...");
             iForEach(ObjectList, i, d->children) {
                 iWidget *child = as_Widget(i.object);
                 if (isArranged_Widget_(child) && ~child->flags & parentCannotResize_WidgetFlag) {
@@ -468,34 +556,52 @@ void arrange_Widget(iWidget *d) {
                         setHeight_Widget_(child, child->flags & unpadded_WidgetFlag ? unpaddedChildSize.y : childSize.y);
                     }
                 }
+                else {
+                    TRACE(d, "child %p cannot be resized (parentCannotResize: %d)", child,
+                          (child->flags & parentCannotResize_WidgetFlag) != 0);
+                }
             }
+            TRACE(d, "...done changing child sizes (EVEN mode)");
         }
     }
     if (d->flags & resizeChildrenToWidestChild_WidgetFlag) {
         const int widest = widestChild_Widget_(d);
+        TRACE(d, "resizing children to widest child (%d)...", widest);
         iForEach(ObjectList, i, d->children) {
             iWidget *child = as_Widget(i.object);
             if (isArranged_Widget_(child) && ~child->flags & parentCannotResize_WidgetFlag) {
                 setWidth_Widget_(child, widest);
             }
+            else {
+                TRACE(d, "child %p cannot be resized (parentCannotResize: %d)", child,
+                      (child->flags & parentCannotResize_WidgetFlag) != 0);
+            }
         }
+        TRACE(d, "...done resizing children to widest child");
     }
     iInt2 pos = initv_I2(d->padding);
+    TRACE(d, "begin positioning children from %d,%d (flags:%s%s)...", pos.x, pos.y,
+          d->flags & arrangeHorizontal_WidgetFlag ? " horiz" : "",
+          d->flags & arrangeVertical_WidgetFlag ? " vert" : "");
     iForEach(ObjectList, i, d->children) {
         iWidget *child = as_Widget(i.object);
-        arrange_Widget(child);
+        arrange_Widget_(child);
         if (!isArranged_Widget_(child)) {
+            TRACE(d, "child %p arranging prohibited", child);
             continue;
         }
         if (child->flags & centerHorizontal_WidgetFlag) {
+            TRACE(d, "child %p is centered, skipping", child);
             continue;
         }
         if (d->flags & (arrangeHorizontal_WidgetFlag | arrangeVertical_WidgetFlag)) {
             if (child->flags &
                 (moveToParentLeftEdge_WidgetFlag | moveToParentRightEdge_WidgetFlag)) {
+                TRACE(d, "child %p is attached an edge, skipping", child);
                 continue; /* Not part of the sequential arrangement .*/
             }
             child->rect.pos = pos;
+            TRACE(d, "child %p set position to %d,%d", child, pos.x, pos.y);
             if (d->flags & arrangeHorizontal_WidgetFlag) {
                 pos.x += child->rect.size.x;
             }
@@ -506,30 +612,20 @@ void arrange_Widget(iWidget *d) {
         else if ((d->flags & resizeChildren_WidgetFlag) == resizeChildren_WidgetFlag &&
                  ~child->flags & moveToParentBottomEdge_WidgetFlag) {
             child->rect.pos = pos;
+            TRACE(d, "child %p set position to %d,%d (not sequential, children being resized)", child, pos.x, pos.y);
         }
         else if (d->flags & resizeWidthOfChildren_WidgetFlag) {
             child->rect.pos.x = pos.x;
+            TRACE(d, "child %p set X to %d (not sequential, children being resized)", child, pos.x);
         }
     }
+    TRACE(d, "...done positioning children");
     /* Update the size of the widget according to the arrangement. */
     if (d->flags & arrangeSize_WidgetFlag) {
-        iRect bounds = zero_Rect();
-        iConstForEach(ObjectList, i, d->children) {
-            const iWidget *child = constAs_Widget(i.object);
-            if (isCollapsed_Widget_(child)) {
-                continue;
-            }
-            iRect childRect = child->rect;
-            if (child->flags & ignoreForParentWidth_WidgetFlag) {
-                childRect.size.x = 0;
-            }
-            if (isEmpty_Rect(bounds)) {
-                bounds = childRect;
-            }
-            else {
-                bounds = union_Rect(bounds, childRect);
-            }
-        }
+        iRect bounds;
+        boundsOfChildren_Widget_(d, &bounds);
+        TRACE(d, "begin arranging own size; bounds of children: %d,%d %dx%d",
+              bounds.pos.x, bounds.pos.y, bounds.size.x, bounds.size.y);
         adjustEdges_Rect(&bounds, -d->padding[1], d->padding[2], d->padding[3], -d->padding[0]);
         if (d->flags & arrangeWidth_WidgetFlag) {
             setWidth_Widget_(d, bounds.size.x);
@@ -539,14 +635,15 @@ void arrange_Widget(iWidget *d) {
                 if (child->flags &
                     (resizeToParentWidth_WidgetFlag |
                      moveToParentLeftEdge_WidgetFlag |
-                     moveToParentBottomEdge_WidgetFlag |
                      moveToParentRightEdge_WidgetFlag)) {
-                    arrange_Widget(child);
+                    TRACE(d, "rearranging child %p because its size or position depends on parent width", child);
+                    arrange_Widget_(child);
                 }
             }
             if (d->flags & moveToParentRightEdge_WidgetFlag) {
                 /* TODO: Fix this: not DRY. See beginning of method. */
                 d->rect.pos.x = width_Rect(innerRect_Widget_(d->parent)) - width_Rect(d->rect);
+                TRACE(d, "after width change moving to right edge of parent, set X to %d", d, d->rect.pos.x);
             }
         }
         if (d->flags & arrangeHeight_WidgetFlag) {
@@ -554,8 +651,10 @@ void arrange_Widget(iWidget *d) {
             /* Parent size changed, must update the children.*/
             iForEach(ObjectList, j, d->children) {
                 iWidget *child = as_Widget(j.object);
-                if (child->flags & resizeToParentHeight_WidgetFlag) {
-                    arrange_Widget(child);
+                if (child->flags & (resizeToParentHeight_WidgetFlag |
+                                    moveToParentBottomEdge_WidgetFlag)) {
+                    TRACE(d, "rearranging child %p because its size or position depends on parent height", child);
+                    arrange_Widget_(child);
                 }
             }
         }
@@ -566,9 +665,31 @@ void arrange_Widget(iWidget *d) {
         if (d->flags & centerHorizontal_WidgetFlag) {
             centerHorizontal_Widget_(d);
         }
+        TRACE(d, "...done arranging own size");
+    }
+    TRACE(d, "END");
+}
+
+void resetSize_Widget(iWidget *d) {
+    if (~d->flags & fixedWidth_WidgetFlag) {
+        d->rect.size.x = d->minSize.x;
+    }
+    if (~d->flags & fixedHeight_WidgetFlag) {
+        d->rect.size.y = d->minSize.y;
+    }
+    iForEach(ObjectList, i, children_Widget(d)) {
+        iWidget *child = as_Widget(i.object);
+        if (isArranged_Widget_(child)) {
+            resetSize_Widget(child);
+        }
     }
 }
 
+void arrange_Widget(iWidget *d) {
+    //resetSize_Widget_(d); /* back to initial default sizes */
+    arrange_Widget_(d);
+}
+
 static void applyVisualOffset_Widget_(const iWidget *d, iInt2 *pos) {
     if (d->flags & (visualOffset_WidgetFlag | dragged_WidgetFlag)) {
         const int off = iRound(value_Anim(&d->visualOffset));
@@ -1354,6 +1475,7 @@ void refresh_Widget(const iAnyObject *d) {
     postRefresh_App();
 }
 
+/* Debug utility for inspecting widget trees. */
 #include "labelwidget.h"
 static void printTree_Widget_(const iWidget *d, int indent) {
     for (int i = 0; i < indent; ++i) {
@@ -1365,9 +1487,17 @@ static void printTree_Widget_(const iWidget *d, int indent) {
                cstr_String(text_LabelWidget((const iLabelWidget *) d)),
                cstr_String(command_LabelWidget((const iLabelWidget *) d)));
     }
-    printf("size:%dx%d [%d..%d %d:%d] flags:%08llx%s\n", d->rect.size.x, d->rect.size.y,
-           d->padding[0], d->padding[2], d->padding[1], d->padding[3],
-           (long long unsigned int) d->flags, d->flags & tight_WidgetFlag ? " tight" : "");
+    printf("size:%dx%d {min:%dx%d} [%d..%d %d:%d] flags:%08llx%s%s%s%s%s\n",
+           d->rect.size.x, d->rect.size.y,
+           d->minSize.x, d->minSize.y,
+           d->padding[0], d->padding[2],
+           d->padding[1], d->padding[3],
+           (long long unsigned int) d->flags,
+           d->flags & expand_WidgetFlag ? " exp" : "",
+           d->flags & tight_WidgetFlag ? " tight" : "",
+           d->flags & fixedWidth_WidgetFlag ? " fixW" : "",
+           d->flags & fixedHeight_WidgetFlag ? " fixH" : "",
+           d->flags & resizeToParentWidth_WidgetFlag ? " rsPrnW" : "");
     iConstForEach(ObjectList, i, d->children) {
         printTree_Widget_(i.object, indent + 1);
     }
diff --git a/src/ui/widget.h b/src/ui/widget.h
index bde13272..4c5b7d85 100644
--- a/src/ui/widget.h
+++ b/src/ui/widget.h
@@ -129,6 +129,7 @@ struct Impl_Widget {
     iString      id;
     int64_t      flags;
     iRect        rect;
+    iInt2        minSize;
     int          padding[4]; /* left, top, right, bottom */
     iAnim        visualOffset;
     int          bgColor;
@@ -221,7 +222,8 @@ iBool   hasParent_Widget            (const iWidget *d, const iWidget *someParent
 void    setId_Widget                (iWidget *, const char *id);
 void    setFlags_Widget             (iWidget *, int64_t flags, iBool set);
 void    setPos_Widget               (iWidget *, iInt2 pos);
-void    setSize_Widget              (iWidget *, iInt2 size);
+void    setFixedSize_Widget         (iWidget *, iInt2 fixedSize);
+void    setMinSize_Widget           (iWidget *, iInt2 minSize);
 void    setPadding_Widget           (iWidget *, int left, int top, int right, int bottom);
 iLocalDef void setPadding1_Widget   (iWidget *d, int padding) { setPadding_Widget(d, padding, padding, padding, padding); }
 void    setVisualOffset_Widget      (iWidget *, int value, uint32_t span, int animFlags);
@@ -238,6 +240,7 @@ iAny *  removeChild_Widget          (iWidget *, iAnyObject *child); /* returns a
 iAny *  child_Widget                (iWidget *, size_t index); /* O(n) */
 size_t  childIndex_Widget           (const iWidget *, const iAnyObject *child); /* O(n) */
 void    arrange_Widget              (iWidget *);
+void    resetSize_Widget            (iWidget *);
 iBool   dispatchEvent_Widget        (iWidget *, const SDL_Event *);
 iBool   processEvent_Widget         (iWidget *, const SDL_Event *);
 void    postCommand_Widget          (const iAnyObject *, const char *cmd, ...);
diff --git a/src/ui/window.c b/src/ui/window.c
index a3675ae5..1b6c5e89 100644
--- a/src/ui/window.c
+++ b/src/ui/window.c
@@ -875,10 +875,10 @@ static void updateMetrics_Window_(iWindow *d) {
         iWidget *appMax   = findChild_Widget(winBar, "winbar.max");
         iWidget *appClose = findChild_Widget(winBar, "winbar.close");
         setPadding_Widget(winBar, 0, gap_UI / 3, 0, 0);
-        setSize_Widget(appMin, init_I2(gap_UI * 11.5f, height_Widget(appTitle)));
-        setSize_Widget(appMax, appMin->rect.size);
-        setSize_Widget(appClose, appMin->rect.size);
-        setSize_Widget(appIcon, init_I2(appIconSize_(), appMin->rect.size.y));
+        setFixedSize_Widget(appMin, init_I2(gap_UI * 11.5f, height_Widget(appTitle)));
+        setFixedSize_Widget(appMax, appMin->rect.size);
+        setFixedSize_Widget(appClose, appMin->rect.size);
+        setFixedSize_Widget(appIcon, init_I2(appIconSize_(), appMin->rect.size.y));
     }
     iWidget *navBar     = findChild_Widget(d->root, "navbar");
     iWidget *lock       = findChild_Widget(navBar, "navbar.lock");
@@ -888,7 +888,7 @@ static void updateMetrics_Window_(iWindow *d) {
     setPadding_Widget(as_Widget(url), 0, gap_UI, 0, gap_UI);
     navBar->rect.size.y = 0; /* recalculate height based on children (FIXME: shouldn't be needed) */
     updateSize_LabelWidget((iLabelWidget *) lock);
-    setSize_Widget(embedPad, init_I2(width_Widget(lock) + gap_UI / 2, 1));
+    setFixedSize_Widget(embedPad, init_I2(width_Widget(lock) + gap_UI / 2, 1));
     setContentPadding_InputWidget((iInputWidget *) url, width_Widget(lock) * 0.75,
                                   width_Widget(lock) * 0.75);
     rightEmbed->rect.pos.y = gap_UI;
@@ -1251,6 +1251,7 @@ static void updateRootSize_Window_(iWindow *d, iBool notifyAlways) {
     const iInt2 oldSize = *size;
     SDL_GetRendererOutputSize(d->render, &size->x, &size->y);
     size->y -= d->keyboardHeight;
+    d->root->minSize = *size;
     if (notifyAlways || !isEqual_I2(oldSize, *size)) {
         const iBool isHoriz = (d->place.lastNotifiedSize.x != size->x);
         const iBool isVert  = (d->place.lastNotifiedSize.y != size->y);
@@ -1465,7 +1466,7 @@ void init_Window(iWindow *d, iRect rect) {
     setupWindow_iOS(d);
 #endif
     d->root = new_Widget();
-    setFlags_Widget(d->root, focusRoot_WidgetFlag, iTrue);
+    setFlags_Widget(d->root, fixedSize_WidgetFlag | focusRoot_WidgetFlag, iTrue);
     d->presentTime = 0.0;
     d->frameTime = SDL_GetTicks();
     d->loadAnimTimer = 0;
@@ -1840,7 +1841,9 @@ iBool processEvent_Window(iWindow *d, const SDL_Event *ev) {
                 insertMacMenus_();
 #endif
                 invalidate_Window_(d);
+                updatePreferencesLayout_Widget(findChild_Widget(d->root, "prefs"));
                 arrange_Widget(d->root);
+                printTree_Widget(findChild_Widget(d->root, "prefs"));
             }
             if (oldHover != hover_Widget()) {
                 postRefresh_App();