Changeset 137 for trunk/src/helpers/dialog.c

Timestamp:

Feb 4, 2002, 6:32:38 PM (24 years ago)

Author:

umoeller

Message:

Buncha fixes, and fixes for fixes.

File:

• trunk/src/helpers/dialog.c (modified) (67 diffs)

trunk/src/helpers/dialog.c

@@ -73 +73 @@
 #include "helpers\xstring.h"
 
+#pragma hdrstop
+
+// #define DEBUG_DIALOG_WINDOWS 1
+
 /*
  *@@category: Helpers\PM helpers\Dialog templates
@@ -109 +113 @@
                                  // created
 
-    const char  *pcszControlsFont;  // from dlghCreateDlg
-
-    // V0.9.14 (2001-08-01) [umoeller]
+    PCSZ        pcszControlsFont;  // from dlghCreateDlg
+
+    // size of the client to be created
+    SIZEL       szlClient;
+
+    // various cached data V0.9.14 (2001-08-01) [umoeller]
     HPS         hps;
-    const char  *pcszFontLast;
+    PCSZ        pcszFontLast;
     LONG        lcidLast;
     FONTMETRICS fmLast;
@@ -190 +197 @@
 typedef struct _TABLEDEF
 {
+    PCOLUMNDEF  pOwningColumn;      // != NULL if this is a nested table
+
     LINKLIST    llRows;             // contains ROWDEF structs, no auto-free
 
@@ -205 +214 @@
 typedef enum _PROCESSMODE
 {
-    PROCESS_CALC_SIZES,             // step 1
-    PROCESS_CALC_POSITIONS,         // step 3
-    PROCESS_CREATE_CONTROLS         // step 4
+    PROCESS_1_CALC_SIZES,             // step 1
+    PROCESS_2_CALC_SIZES_FROM_TABLES, // step 2
+    PROCESS_3_CALC_FINAL_TABLE_SIZES, // step 3
+    PROCESS_4_CALC_POSITIONS,         // step 4
+    PROCESS_5_CREATE_CONTROLS         // step 5
 } PROCESSMODE;
 
@@ -217 +228 @@
 
 #define PM_GROUP_SPACING_X          16
-#define PM_GROUP_SPACING_TOP        20
+#define PM_GROUP_SPACING_TOP        16
 
 APIRET ProcessTable(PTABLEDEF pTableDef,
@@ -293 +304 @@
  *@@changed V0.9.14 (2001-08-01) [umoeller]: now caching fonts, which is significantly faster
  *@@changed V0.9.16 (2001-10-15) [umoeller]: added APIRET
+ *@@changed V0.9.16 (2002-02-02) [umoeller]: added ulWidth
  */
 
 APIRET CalcAutoSizeText(PCONTROLDEF pControlDef,
                         BOOL fMultiLine,          // in: if TRUE, multiple lines
+                        ULONG ulWidth,            // in: proposed width of control
                         PSIZEL pszlAuto,          // out: computed size
                         PDLGPRIVATE pDlgData)
@@ -317 +330 @@
         {
             RECTL rcl = {0, 0, 0, 0};
+            /*
             if (pControlDef->szlControlProposed.cx > 0)
                 rcl.xRight = pControlDef->szlControlProposed.cx;   // V0.9.12 (2001-05-31) [umoeller]
             else
                 rcl.xRight = winhQueryScreenCX() * 2 / 3;
+            */
+            rcl.xRight = ulWidth;
             if (pControlDef->szlControlProposed.cy > 0)
                 rcl.yTop = pControlDef->szlControlProposed.cy;   // V0.9.12 (2001-05-31) [umoeller]
@@ -358 +374 @@
 
 APIRET CalcAutoSize(PCONTROLDEF pControlDef,
+                    ULONG ulWidth,            // in: proposed width of control
                     PSIZEL pszlAuto,          // out: computed size
                     PDLGPRIVATE pDlgData)
@@ -372 +389 @@
             if (!(arc = CalcAutoSizeText(pControlDef,
                                          FALSE,         // no multiline
+                                         ulWidth,
                                          pszlAuto,
                                          pDlgData)))
@@ -404 +422 @@
                 arc = CalcAutoSizeText(pControlDef,
                                        ((pControlDef->flStyle & DT_WORDBREAK) != 0),
+                                       ulWidth,
                                        pszlAuto,
                                        pDlgData);
@@ -437 +456 @@
             pszlAuto->cy =   pDlgData->fmLast.lMaxBaselineExt
                            + pDlgData->fmLast.lExternalLeading
-                           + 5;         // some space
+                           + 7;         // some space
     }
 
@@ -445 +464 @@
 /*
  *@@ ColumnCalcSizes:
- *      implementation for PROCESS_CALC_SIZES in
+ *      implementation for PROCESS_1_CALC_SIZES in
  *      ProcessColumn.
+ *
+ *      This gets called a second time for
+ *      PROCESS_3_CALC_FINAL_TABLE_SIZES (V0.9.16).
  *
  *@@added V0.9.15 (2001-08-26) [umoeller]
@@ -455 +477 @@
 
 APIRET ColumnCalcSizes(PCOLUMNDEF pColumnDef,
+                       PROCESSMODE ProcessMode,     // in: PROCESS_1_CALC_SIZES or PROCESS_3_CALC_FINAL_TABLE_SIZES
                        PDLGPRIVATE pDlgData)
 {
     APIRET arc = NO_ERROR;
 
-    ULONG       ulXSpacing = 0,
-                ulYSpacing = 0;
+    ULONG       ulExtraCX = 0,
+                ulExtraCY = 0;
     if (pColumnDef->fIsNestedTable)
     {
@@ -467 +490 @@
         if (!(arc = ProcessTable(pTableDef,
                                  NULL,
-                                 PROCESS_CALC_SIZES,
+                                 ProcessMode,
                                  pDlgData)))
         {
@@ -491 +514 @@
 
                 // in any case, make this wider
-                ulXSpacing =    2 * PM_GROUP_SPACING_X;
-                ulYSpacing =    // 3 * PM_GROUP_SPACING_X;
-                            (PM_GROUP_SPACING_X + PM_GROUP_SPACING_TOP);
+                ulExtraCX =    2 * PM_GROUP_SPACING_X;
+                ulExtraCY =    (PM_GROUP_SPACING_X + PM_GROUP_SPACING_TOP);
             }
         }
@@ -501 +523 @@
         // no nested table, but control:
         PCONTROLDEF pControlDef = (PCONTROLDEF)pColumnDef->pvDefinition;
-        PSIZEL      pszl = &pControlDef->szlControlProposed;
         SIZEL       szlAuto;
 
-        if (    (pszl->cx < -1)
-             && (pszl->cx >= -100)
-           )
-        {
-            // other negative CX value:
-            // this is then a percentage of the row width... ignore for now
-            // V0.9.16 (2002-02-02) [umoeller]
-            szlAuto.cx = 0;
-            szlAuto.cy = 0;
-        }
-        else if (    (pszl->cx == -1)
-                  || (pszl->cy == -1)
-                )
-        {
-            arc = CalcAutoSize(pControlDef,
-                               &szlAuto,
-                               pDlgData);
-        }
-
-        if (!arc)
-        {
-            if (pszl->cx < 0)
-                pColumnDef->cpControl.cx = szlAuto.cx;
-            else
-                pColumnDef->cpControl.cx = pszl->cx;
-
-            if (pszl->cy < 0)
-                pColumnDef->cpControl.cy = szlAuto.cy;
-            else
-                pColumnDef->cpControl.cy = pszl->cy;
-
-            // @@todo hack sizes
-
-            ulXSpacing
-            = ulYSpacing
-            = (2 * pControlDef->ulSpacing);
-        }
+        // do auto-size calculations only on the first loop
+        // V0.9.16 (2002-02-02) [umoeller]
+        if (ProcessMode == PROCESS_1_CALC_SIZES)
+        {
+            if (    (pControlDef->szlControlProposed.cx == -1)
+                 || (pControlDef->szlControlProposed.cy == -1)
+               )
+            {
+                ULONG ulWidth;
+                if (pControlDef->szlControlProposed.cx == -1)
+                    ulWidth = 1000;
+                else
+                    ulWidth = pControlDef->szlControlProposed.cx;
+                arc = CalcAutoSize(pControlDef,
+                                   ulWidth,
+                                   &szlAuto,
+                                   pDlgData);
+            }
+
+            if (    (pControlDef->szlControlProposed.cx < -1)
+                 && (pControlDef->szlControlProposed.cx >= -100)
+               )
+            {
+                // other negative CX value:
+                // this is then a percentage of the table width... ignore for now
+                // V0.9.16 (2002-02-02) [umoeller]
+                szlAuto.cx = 0;
+            }
+
+            if (    (pControlDef->szlControlProposed.cy < -1)
+                 && (pControlDef->szlControlProposed.cy >= -100)
+               )
+            {
+                // other negative CY value:
+                // this is then a percentage of the row height... ignore for now
+                // V0.9.16 (2002-02-02) [umoeller]
+                szlAuto.cy = 0;
+            }
+
+            if (!arc)
+            {
+                if (pControlDef->szlControlProposed.cx < 0)
+                    pColumnDef->cpControl.cx = szlAuto.cx;
+                else
+                    pColumnDef->cpControl.cx = pControlDef->szlControlProposed.cx;
+
+                if (pControlDef->szlControlProposed.cy < 0)
+                    pColumnDef->cpControl.cy = szlAuto.cy;
+                else
+                    pColumnDef->cpControl.cy = pControlDef->szlControlProposed.cy;
+            }
+
+        } // end if (ProcessMode == PROCESS_1_CALC_SIZES)
+
+
+        ulExtraCX
+        = ulExtraCY
+        = (2 * pControlDef->ulSpacing);
     }
 
     pColumnDef->cpColumn.cx =   pColumnDef->cpControl.cx
-                               + ulXSpacing;
+                               + ulExtraCX;
     pColumnDef->cpColumn.cy =   pColumnDef->cpControl.cy
-                               + ulYSpacing;
+                               + ulExtraCY;
 
     return (arc);
@@ -553 +595 @@
 /*
  *@@ ColumnCalcPositions:
- *      implementation for PROCESS_CALC_POSITIONS in
+ *      implementation for PROCESS_4_CALC_POSITIONS in
  *      ProcessColumn.
  *
@@ -562 +604 @@
 APIRET ColumnCalcPositions(PCOLUMNDEF pColumnDef,
                            PROWDEF pOwningRow,          // in: current row from ProcessRow
-                           PLONG plX,                   // in/out: PROCESS_CALC_POSITIONS only
+                           PLONG plX,                   // in/out: PROCESS_4_CALC_POSITIONS only
                            PDLGPRIVATE pDlgData)
 {
@@ -568 +610 @@
 
     // calculate column position: this includes spacing
-    ULONG ulSpacing = 0;
+    LONG   lSpacingX = 0,
+           lSpacingY = 0;
 
     // column position = *plX on ProcessRow stack
@@ -599 +642 @@
         // should we create a PM control around the table?
         if (pTableDef->pCtlDef)
+        {
             // yes:
-            ulSpacing = PM_GROUP_SPACING_X / 2;     // V0.9.16 (2001-10-15) [umoeller]
+            lSpacingX = PM_GROUP_SPACING_X;     // V0.9.16 (2001-10-15) [umoeller]
+            lSpacingY = PM_GROUP_SPACING_X;     // V0.9.16 (2001-10-15) [umoeller]
+        }
     }
     else
@@ -606 +652 @@
         // no nested table, but control:
         PCONTROLDEF pControlDef = (PCONTROLDEF)pColumnDef->pvDefinition;
-        ulSpacing = pControlDef->ulSpacing;
+        lSpacingX = lSpacingY = pControlDef->ulSpacing;
     }
 
@@ -613 +659 @@
 
     // calculate CONTROL pos from COLUMN pos by applying spacing
-    pColumnDef->cpControl.x =   pColumnDef->cpColumn.x
-                              + ulSpacing;
-    pColumnDef->cpControl.y =   pColumnDef->cpColumn.y
-                              + ulSpacing;
+    pColumnDef->cpControl.x =   (LONG)pColumnDef->cpColumn.x
+                              + lSpacingX;
+    pColumnDef->cpControl.y =   (LONG)pColumnDef->cpColumn.y
+                              + lSpacingY;
 
     if (pColumnDef->fIsNestedTable)
@@ -626 +672 @@
         arc = ProcessTable(pTableDef,
                            &pColumnDef->cpControl,   // start pos for new table
-                           PROCESS_CALC_POSITIONS,
+                           PROCESS_4_CALC_POSITIONS,
                            pDlgData);
     }
@@ -635 +681 @@
 /*
  *@@ ColumnCreateControls:
- *      implementation for PROCESS_CREATE_CONTROLS in
+ *      implementation for PROCESS_5_CREATE_CONTROLS in
  *      ProcessColumn.
  *
@@ -648 +694 @@
     APIRET      arc = NO_ERROR;
 
-    PCONTROLPOS pcp = NULL;
     PCONTROLDEF pControlDef = NULL;
-    const char  *pcszTitle = NULL;
+
+    PCSZ        pcszClass = NULL;
+    PCSZ        pcszTitle = NULL;
     ULONG       flStyle = 0;
     LHANDLE     lHandleSet = NULLHANDLE;
@@ -665 +712 @@
         if (!(arc = ProcessTable(pTableDef,
                                  NULL,
-                                 PROCESS_CREATE_CONTROLS,
+                                 PROCESS_5_CREATE_CONTROLS,
                                  pDlgData)))
         {
@@ -674 +721 @@
             {
                 // yes:
-                pcp  = &pColumnDef->cpColumn;  // !! not control
+                // pcp  = &pColumnDef->cpColumn;  // !! not control
                 pControlDef = pTableDef->pCtlDef;
+                pcszClass = pControlDef->pcszClass;
                 pcszTitle = pControlDef->pcszText;
                 flStyle = pControlDef->flStyle;
 
-                x = pcp->x + pDlgData->ptlTotalOfs.x;
-                cx = pcp->cx - PM_GROUP_SPACING_X;
+                x  =   pColumnDef->cpColumn.x
+                     + pDlgData->ptlTotalOfs.x
+                     + PM_GROUP_SPACING_X / 2;
+                cx =   pColumnDef->cpColumn.cx
+                     - PM_GROUP_SPACING_X;
                     // note, just one spacing: for the _column_ size,
                     // we have specified 2 X spacings
-                y = pcp->y + pDlgData->ptlTotalOfs.y;
+                y  =   pColumnDef->cpColumn.y
+                     + pDlgData->ptlTotalOfs.y
+                     + PM_GROUP_SPACING_X / 2;
                 // cy = pcp->cy - PM_GROUP_SPACING_X;
-                cy = pcp->cy - /* PM_GROUP_SPACING_X - */ PM_GROUP_SPACING_TOP / 2;
+                // cy = pcp->cy - /* PM_GROUP_SPACING_X - */ PM_GROUP_SPACING_TOP;
+                cy =   pColumnDef->cpColumn.cy
+                     - PM_GROUP_SPACING_X / 2; //  - PM_GROUP_SPACING_TOP / 2;
             }
+
+#ifdef DEBUG_DIALOG_WINDOWS
+            {
+                HWND hwndDebug;
+                // debug: create a frame with the exact size
+                // of the _column_ (not the control), so this
+                // includes spacing
+                hwndDebug =
+                   WinCreateWindow(pDlgData->hwndDlg,   // parent
+                                WC_STATIC,
+                                "",
+                                WS_VISIBLE | SS_FGNDFRAME,
+                                pTableDef->cpTable.x + pDlgData->ptlTotalOfs.x,
+                                pTableDef->cpTable.y + pDlgData->ptlTotalOfs.y,
+                                pTableDef->cpTable.cx,
+                                pTableDef->cpTable.cy,
+                                pDlgData->hwndDlg,   // owner
+                                HWND_BOTTOM,
+                                -1,
+                                NULL,
+                                NULL);
+                winhSetPresColor(hwndDebug, PP_FOREGROUNDCOLOR, RGBCOL_BLUE);
+            }
+#endif
         }
     }
@@ -693 +772 @@
         // no nested table, but control:
         pControlDef = (PCONTROLDEF)pColumnDef->pvDefinition;
-        pcp = &pColumnDef->cpControl;
+        // pcp = &pColumnDef->cpControl;
+        pcszClass = pControlDef->pcszClass;
         pcszTitle = pControlDef->pcszText;
         flStyle = pControlDef->flStyle;
 
-        x = pcp->x + pDlgData->ptlTotalOfs.x;
-        cx = pcp->cx;
-        y = pcp->y + pDlgData->ptlTotalOfs.y;
-        cy = pcp->cy;
+        x  =   pColumnDef->cpControl.x
+             + pDlgData->ptlTotalOfs.x;
+        cx =   pColumnDef->cpControl.cx;
+        y  =   pColumnDef->cpControl.y
+             + pDlgData->ptlTotalOfs.y;
+        cy =   pColumnDef->cpControl.cy;
 
         // now implement hacks for certain controls
@@ -734 +816 @@
 
             case 0xffff0006L:   // entry field
+            case 0xffff000AL:   // MLE:
                 // the stupid entry field resizes itself if it has
                 // the ES_MARGIN style, so correlate that too... dammit
@@ -743 +826 @@
 
                     x += cxMargin;
-                    y += cxMargin;
+                    y += cyMargin;
+                    cx -= 2 * cxMargin;
+                    cy -= 2 * cyMargin;
+                    // cy -= cxMargin;
                 }
             break;
@@ -749 +835 @@
     }
 
-    if (pcp && pControlDef)
+    if (pControlDef)
     {
         // create something:
-        // PPRESPARAMS ppp = NULL;
-
-        const char  *pcszFont = pControlDef->pcszFont;
+        PCSZ pcszFont = pControlDef->pcszFont;
                         // can be NULL, or CTL_COMMON_FONT
         if (pcszFont == CTL_COMMON_FONT)
             pcszFont = pDlgData->pcszControlsFont;
 
-        /* if (pcszFont)
-            winhStorePresParam(&ppp,
-                               PP_FONTNAMESIZE,
-                               strlen(pcszFont),
-                               (PVOID)pcszFont); */
-
         if (pColumnDef->hwndControl
             = WinCreateWindow(pDlgData->hwndDlg,   // parent
-                              (PSZ)pControlDef->pcszClass,
+                              (PSZ)pcszClass,   // hacked
                               (pcszTitle)   // hacked
                                     ? (PSZ)pcszTitle
@@ -782 +860 @@
                               NULL))
         {
+#ifdef DEBUG_DIALOG_WINDOWS
+            {
+                HWND hwndDebug;
+                // debug: create a frame with the exact size
+                // of the _column_ (not the control), so this
+                // includes spacing
+                hwndDebug =
+                   WinCreateWindow(pDlgData->hwndDlg,   // parent
+                                WC_STATIC,
+                                "",
+                                WS_VISIBLE | SS_FGNDFRAME,
+                                pColumnDef->cpColumn.x + pDlgData->ptlTotalOfs.x,
+                                pColumnDef->cpColumn.y + pDlgData->ptlTotalOfs.y,
+                                pColumnDef->cpColumn.cx,
+                                pColumnDef->cpColumn.cy,
+                                pDlgData->hwndDlg,   // owner
+                                HWND_BOTTOM,
+                                -1,
+                                NULL,
+                                NULL);
+                winhSetPresColor(hwndDebug, PP_FOREGROUNDCOLOR, RGBCOL_RED);
+            }
+#endif
+
             if (lHandleSet)
             {
@@ -850 +952 @@
  *      This does the following:
  *
- *      -- PROCESS_CALC_SIZES: size is taken from control def,
+ *      -- PROCESS_1_CALC_SIZES: size is taken from control def,
  *         or for tables, this produces a recursive ProcessTable
  *         call.
  *         Preconditions: none.
  *
- *      -- PROCESS_CALC_POSITIONS: position of each column
+ *      -- PROCESS_4_CALC_POSITIONS: position of each column
  *         is taken from *plX, which is increased by the
  *         column width by this call.
@@ -865 +967 @@
  *         size here.
  *
- *      -- PROCESS_CREATE_CONTROLS: well, creates the controls.
+ *      -- PROCESS_5_CREATE_CONTROLS: well, creates the controls.
  *
  *         For tables, this recurses again. If the table has
@@ -878 +980 @@
                      PROWDEF pOwningRow,          // in: current row from ProcessRow
                      PROCESSMODE ProcessMode,     // in: processing mode (see ProcessAll)
-                     PLONG plX,                   // in/out: PROCESS_CALC_POSITIONS only
+                     PLONG plX,                   // in/out: PROCESS_4_CALC_POSITIONS only
                      PDLGPRIVATE pDlgData)
 {
@@ -888 +990 @@
     {
         /*
-         * PROCESS_CALC_SIZES:
+         * PROCESS_1_CALC_SIZES:
          *      step 1.
          */
 
-        case PROCESS_CALC_SIZES:
+        case PROCESS_1_CALC_SIZES:
             arc = ColumnCalcSizes(pColumnDef,
+                                  ProcessMode,
                                   pDlgData);
         break;
 
         /*
-         * PROCESS_CALC_POSITIONS:
-         *      step 2.
+         * PROCESS_2_CALC_SIZES_FROM_TABLES:
+         *
          */
 
-        case PROCESS_CALC_POSITIONS:
+        case PROCESS_2_CALC_SIZES_FROM_TABLES:
+            if (pColumnDef->fIsNestedTable)
+            {
+                PTABLEDEF pTableDef = (PTABLEDEF)pColumnDef->pvDefinition;
+                if (!(arc = ProcessTable(pTableDef,
+                                         NULL,
+                                         PROCESS_2_CALC_SIZES_FROM_TABLES,
+                                         pDlgData)))
+                    ;
+            }
+            else
+            {
+                // no nested table, but control:
+                PCONTROLDEF pControlDef = (PCONTROLDEF)pColumnDef->pvDefinition;
+
+                if (    (pControlDef->szlControlProposed.cx < -1)
+                     && (pControlDef->szlControlProposed.cx >= -100)
+                   )
+                {
+                    // other negative CX value:
+                    // this we ignored during PROCESS_1_CALC_SIZES
+                    // (see ColumnCalcSizes); now set it to the
+                    // table width!
+                    ULONG cxThis = pOwningRow->pOwningTable->cpTable.cx
+                                    * -pControlDef->szlControlProposed.cx / 100;
+
+                    // but the table already has spacing applied,
+                    // so reduce that
+                    pColumnDef->cpControl.cx = cxThis
+                                            - (2 * pControlDef->ulSpacing);
+
+                    pColumnDef->cpColumn.cx = cxThis;
+
+                    // now we might have to re-compute auto-size
+                    if (pControlDef->szlControlProposed.cy == -1)
+                    {
+                        SIZEL   szlAuto;
+                        if (!(arc = CalcAutoSize(pControlDef,
+                                                 // now that we now the width,
+                                                 // use that!
+                                                 pColumnDef->cpControl.cx,
+                                                 &szlAuto,
+                                                 pDlgData)))
+                        {
+                            LONG cyColumnOld = pColumnDef->cpColumn.cy;
+                            LONG lDelta;
+                            PROWDEF pRowThis;
+
+                            pColumnDef->cpControl.cy = szlAuto.cy;
+                            pColumnDef->cpColumn.cy = szlAuto.cy
+                                        + (2 * pControlDef->ulSpacing);
+                        }
+                    }
+                }
+
+                if (    (pControlDef->szlControlProposed.cy < -1)
+                     && (pControlDef->szlControlProposed.cy >= -100)
+                   )
+                {
+                    // same thing for CY, but this time we
+                    // take the row height
+                    ULONG cyThis = pOwningRow->cpRow.cy
+                                    * -pControlDef->szlControlProposed.cy / 100;
+
+                    // but the table already has spacing applied,
+                    // so reduce that
+                    pColumnDef->cpControl.cy = cyThis
+                                            - (2 * pControlDef->ulSpacing);
+
+                    pColumnDef->cpColumn.cy = cyThis;
+                }
+            }
+        break;
+
+        /*
+         * PROCESS_3_CALC_FINAL_TABLE_SIZES:
+         *
+         */
+
+        case PROCESS_3_CALC_FINAL_TABLE_SIZES:
+            // re-run calc sizes since we now know all
+            // the auto-size items
+            arc = ColumnCalcSizes(pColumnDef,
+                                  ProcessMode,
+                                  pDlgData);
+        break;
+
+        /*
+         * PROCESS_4_CALC_POSITIONS:
+         *      step 4.
+         */
+
+        case PROCESS_4_CALC_POSITIONS:
             arc = ColumnCalcPositions(pColumnDef,
                                       pOwningRow,
@@ -910 +1105 @@
 
         /*
-         * PROCESS_CREATE_CONTROLS:
-         *      step 3.
+         * PROCESS_5_CREATE_CONTROLS:
+         *      step 5.
          */
 
-        case PROCESS_CREATE_CONTROLS:
+        case PROCESS_5_CREATE_CONTROLS:
             arc = ColumnCreateControls(pColumnDef,
                                        pDlgData);
@@ -944 +1139 @@
     pRowDef->pOwningTable = pOwningTable;
 
-    if (ProcessMode == PROCESS_CALC_SIZES)
+    if (    (ProcessMode == PROCESS_1_CALC_SIZES)
+         || (ProcessMode == PROCESS_3_CALC_FINAL_TABLE_SIZES)
+       )
     {
         pRowDef->cpRow.cx = 0;
         pRowDef->cpRow.cy = 0;
     }
-    else if (ProcessMode == PROCESS_CALC_POSITIONS)
+    else if (ProcessMode == PROCESS_4_CALC_POSITIONS)
     {
         // set up x and y so that the columns can
@@ -969 +1166 @@
         if (!(arc = ProcessColumn(pColumnDefThis, pRowDef, ProcessMode, &lX, pDlgData)))
         {
-            if (ProcessMode == PROCESS_CALC_SIZES)
+            if (    (ProcessMode == PROCESS_1_CALC_SIZES)
+                 || (ProcessMode == PROCESS_3_CALC_FINAL_TABLE_SIZES)
+               )
             {
                 // row width = sum of all columns
@@ -997 +1196 @@
  *      nested table is found in a COLUMNDEF.
  *
- *      With PROCESS_CALC_POSITIONS, pptl must specify
+ *      With PROCESS_4_CALC_POSITIONS, pptl must specify
  *      the lower left corner of the table. For the
  *      root call, this will be {0, 0}; for nested calls,
@@ -1006 +1205 @@
 
 APIRET ProcessTable(PTABLEDEF pTableDef,
-                    const CONTROLPOS *pcpTable,       // in: table position with PROCESS_CALC_POSITIONS
+                    const CONTROLPOS *pcpTable,       // in: table position with PROCESS_4_CALC_POSITIONS
                     PROCESSMODE ProcessMode,          // in: processing mode (see ProcessAll)
                     PDLGPRIVATE pDlgData)
@@ -1014 +1213 @@
     PLISTNODE pNode;
 
-    if (ProcessMode == PROCESS_CALC_SIZES)
-    {
-        pTableDef->cpTable.cx = 0;
-        pTableDef->cpTable.cy = 0;
-    }
-    else if (ProcessMode == PROCESS_CALC_POSITIONS)
-    {
-        pTableDef->cpTable.x = pcpTable->x;
-        pTableDef->cpTable.y = pcpTable->y;
-
-        // start the rows on top
-        lY = pcpTable->y + pTableDef->cpTable.cy;
+    switch (ProcessMode)
+    {
+        case PROCESS_1_CALC_SIZES:
+        case PROCESS_3_CALC_FINAL_TABLE_SIZES:
+            pTableDef->cpTable.cx = 0;
+            pTableDef->cpTable.cy = 0;
+        break;
+
+        case PROCESS_4_CALC_POSITIONS:
+            pTableDef->cpTable.x = pcpTable->x;
+            pTableDef->cpTable.y = pcpTable->y;
+
+            // start the rows on top
+            lY = pcpTable->y + pTableDef->cpTable.cy;
+        break;
     }
 
@@ -1034 +1236 @@
         if (!(arc = ProcessRow(pRowDefThis, pTableDef, ProcessMode, &lY, pDlgData)))
         {
-            if (ProcessMode == PROCESS_CALC_SIZES)
+            if (    (ProcessMode == PROCESS_1_CALC_SIZES)
+                 || (ProcessMode == PROCESS_3_CALC_FINAL_TABLE_SIZES)
+               )
             {
                 // table width = maximum width of a row
@@ -1064 +1268 @@
  *      following for each call (in this order):
  *
- *      -- PROCESS_CALC_SIZES: calculates the sizes
- *         of all tables, rows, columns, and controls.
- *
- *         After this first call, we know all the sizes
- *         only and can then calculate the positions.
- *
- *      -- PROCESS_CALC_POSITIONS: calculates the positions
- *         based on the sizes calculated before.
- *
- *      -- PROCESS_CREATE_CONTROLS: creates the controls with the
- *         positions and sizes calculated before.
+ *      --  PROCESS_1_CALC_SIZES: calculates the preliminary
+ *          sizes of all tables, rows, columns, and controls
+ *          except those controls that have specified that
+ *          their size should depend on others.
+ *
+ *      --  PROCESS_2_CALC_SIZES_FROM_TABLES: calculates the
+ *          sizes of those controls that want to depend on
+ *          others.
+ *
+ *      --  PROCESS_3_CALC_FINAL_TABLE_SIZES: since the table
+ *          and row sizes might have changed during
+ *          PROCESS_2_CALC_SIZES_FROM_TABLES, we need to re-run
+ *          to re-calculate the size of all rows and tables.
+ *          After this first call, we know _all_ the sizes
+ *          and can then calculate the positions.
+ *
+ *      --  PROCESS_4_CALC_POSITIONS: calculates the positions
+ *          based on the sizes calculated before.
+ *
+ *      --  PROCESS_5_CREATE_CONTROLS: creates the controls with the
+ *          positions and sizes calculated before.
  *
  *      The second trick is the precondition that tables may
@@ -1084 +1298 @@
 
 APIRET ProcessAll(PDLGPRIVATE pDlgData,
-                  PSIZEL pszlClient,
                   PROCESSMODE ProcessMode)
 {
@@ -1094 +1307 @@
     switch (ProcessMode)
     {
-        case PROCESS_CALC_SIZES:
-            pszlClient->cx = 0;
-            pszlClient->cy = 0;
+        case PROCESS_1_CALC_SIZES:
+        case PROCESS_3_CALC_FINAL_TABLE_SIZES:
+            pDlgData->szlClient.cx = 0;
+            pDlgData->szlClient.cy = 0;
         break;
 
-        case PROCESS_CALC_POSITIONS:
+        case PROCESS_4_CALC_POSITIONS:
             // start with the table on top
-            cpTable.y = pszlClient->cy;
+            cpTable.y = pDlgData->szlClient.cy;
         break;
     }
@@ -1109 +1323 @@
         PTABLEDEF pTableDefThis = (PTABLEDEF)pNode->pItemData;
 
-        if (ProcessMode == PROCESS_CALC_POSITIONS)
+        if (ProcessMode == PROCESS_4_CALC_POSITIONS)
         {
             cpTable.x = 0;
@@ -1120 +1334 @@
                                  pDlgData)))
         {
-            if (ProcessMode == PROCESS_CALC_SIZES)
+            if (    (ProcessMode == PROCESS_2_CALC_SIZES_FROM_TABLES)
+                 || (ProcessMode == PROCESS_3_CALC_FINAL_TABLE_SIZES)
+               )
             {
                 // all sizes have now been computed:
-                pszlClient->cx += pTableDefThis->cpTable.cx;
-                pszlClient->cy += pTableDefThis->cpTable.cy;
+                pDlgData->szlClient.cx += pTableDefThis->cpTable.cx;
+                pDlgData->szlClient.cy += pTableDefThis->cpTable.cy;
             }
         }
@@ -1335 +1551 @@
                                                  pCurrentTable,
                                                  &pColumnDef)))
+                        {
+                            pCurrentTable->pOwningColumn = pColumnDef;
                             lstAppendItem(&pCurrentRow->llColumns,
                                           pColumnDef);
+                        }
                     }
                 }
@@ -1427 +1646 @@
  *@@ Dlg2_CalcSizes:
  *
+ *      After this, DLGPRIVATE.szlClient is valid.
+ *
  *@@added V0.9.15 (2001-08-26) [umoeller]
  */
 
-APIRET Dlg2_CalcSizes(PDLGPRIVATE pDlgData,
-                      PSIZEL pszlClient)          // out: dialog's client size
-{
-    APIRET arc = ProcessAll(pDlgData,
-                            pszlClient,
-                            PROCESS_CALC_SIZES);
+APIRET Dlg2_CalcSizes(PDLGPRIVATE pDlgData)
+{
+    APIRET arc;
+
+    if (!(arc = ProcessAll(pDlgData,
+                           PROCESS_1_CALC_SIZES)))
                      // this goes into major recursions...
+        // run again to compute sizes that depend on tables
+        if (!(arc = ProcessAll(pDlgData,
+                               PROCESS_2_CALC_SIZES_FROM_TABLES)))
+            arc = ProcessAll(pDlgData,
+                             PROCESS_3_CALC_FINAL_TABLE_SIZES);
 
     // free the cached font resources that
@@ -1461 +1687 @@
 
 APIRET Dlg3_PositionAndCreate(PDLGPRIVATE pDlgData,
-                              PSIZEL pszlClient,          // in: dialog's client size
                               HWND *phwndFocusItem)       // out: item to give focus to
 {
@@ -1472 +1697 @@
 
     ProcessAll(pDlgData,
-               pszlClient,
-               PROCESS_CALC_POSITIONS);
+               PROCESS_4_CALC_POSITIONS);
 
     /*
@@ -1485 +1709 @@
 
     ProcessAll(pDlgData,
-               pszlClient,
-               PROCESS_CREATE_CONTROLS);
+               PROCESS_5_CREATE_CONTROLS);
 
     if (pDlgData->hwndDefPushbutton)
@@ -1564 +1787 @@
  *      automatically here. Even better, for many controls,
  *      auto-sizing is supported according to the control's
- *      text (e.g. for statics and checkboxes).
+ *      text (e.g. for statics and checkboxes). In a way,
+ *      this is a bit similar to HTML tables.
  *
  *      A regular standard dialog would use something like
@@ -1600 +1824 @@
  *      in some table. Tables may also nest (see below).
  *
- *      The macros are:
+ *      The DLGHITEM macros are:
  *
  *      --  START_TABLE starts a new table. The tables may nest,
@@ -1653 +1877 @@
  *          always be a START_ROW first.
  *
+ *      While it is possible to set up the CONTROLDEFs manually
+ *      as static structures, I recommend using the bunch of
+ *      other macros that were defined in dialog.h for this.
+ *      For example, you can use CONTROLDEF_PUSHBUTTON to create
+ *      a push button, and many more.
+ *
  *      To create a dialog, set up arrays like the following:
  *
  +          // control definitions referenced by DlgTemplate:
  +          CONTROLDEF
- +      (1)             GroupDef = {
- +                                  WC_STATIC, "", ....,
- +                                  { 0, 0 },       // size, ignored for groups
- +                                  5               // spacing
- +                               },
- +      (2)             CnrDef = {
- +                                  WC_CONTAINER, "", ....,
- +                                  { 50, 50 },     // size
- +                                  5               // spacing
- +                               },
- +      (3)             Static = {
- +                                  WC_STATIC, "Static below cnr", ...,
- +                                  { SZL_AUTOSIZE, SZL_AUTOSIZE },     // size
- +                                  5               // spacing
- +                               },
- +      (4)             OKButton = {
- +                                  WC_STATIC, "~OK", ...,
- +                                  { 100, 30 },    // size
- +                                  5               // spacing
- +                               },
- +      (5)             CancelButton = {
- +                                  WC_STATIC, "~Cancel", ...,
- +                                  { 100, 30 },    // size
- +                                  5               // spacing
- +                               };
+ +      (1)             GroupDef = CONTROLDEF_GROUP("Group",
+ +                                                  -1,     // ID
+ +                                                  SZL_AUTOSIZE,
+ +                                                  SZL_AUTOSIZE),
+ +      (2)             CnrDef = CONTROLDEF_CONTAINER(-1,   // ID,
+ +                                                  50,
+ +                                                  50),
+ +      (3)             Static = CONTROLDEF_TEXT("Static below cnr",
+ +                                                  -1,     // ID
+ +                                                  SZL_AUTOSIZE,
+ +                                                  SZL_AUTOSIZE),
+ +      (4)             OKButton = CONTROLDEF_DEFPUSHBUTTON("~OK",
+ +                                                  DID_OK,
+ +                                                  SZL_AUTOSIZE,
+ +                                                  SZL_AUTOSIZE),
+ +      (5)             CancelButton = CONTROLDEF_PUSHBUTTON("~Cancel",
+ +                                                  DID_CANCEL,
+ +                                                  SZL_AUTOSIZE,
+ +                                                  SZL_AUTOSIZE);
  +
  +          DLGHITEM DlgTemplate[] =
@@ -1723 +1947 @@
  +          ÈÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍÍŒ
  *
- *      <B>Errors:</B>
- *
- *      This does not return a HWND, but an APIRET. This will be
- *      one of the following:
- *
- *      --  NO_ERROR: only in that case, the phwndDlg ptr
- *          receives the HWND of the new dialog, which can
- *          then be given to WinProcessDlg. Don't forget
- *          WinDestroyWindow.
- *
- *      --  ERROR_NOT_ENOUGH_MEMORY
- *
- *      --  DLGERR_ROW_BEFORE_TABLE: a row definition appeared
- *          outside a table definition.
- *
- *      --  DLGERR_CONTROL_BEFORE_ROW: a control definition
- *          appeared right after a table definition. You must
- *          specify a row first.
- *
- *      --  DLGERR_NULL_CTL_DEF: TYPE_END_TABLE was specified,
- *          but the CONTROLDEF ptr was NULL.
- *
- *      --  DLGERR_CANNOT_CREATE_FRAME: unable to create the
- *          WC_FRAME window. Maybe an invalid owner was specified.
- *
- *      --  DLGERR_INVALID_CODE: invalid "Type" field in
- *          DLGHITEM.
- *
- *      --  DLGERR_TABLE_NOT_CLOSED, DLGERR_TOO_MANY_TABLES_CLOSED:
- *          improper nesting of TYPE_START_NEW_TABLE and
- *          TYPE_END_TABLE fields.
- *
  *      <B>Example:</B>
  *
@@ -1762 +1954 @@
  +          if (NO_ERROR == dlghCreateDlg(&hwndDlg,
  +                                        hwndOwner,
- +                                        FCF_DLGBORDER | FCF_NOBYTEALIGN,
+ +                                        FCF_TITLEBAR | FCF_SYSMENU
+ +                                           | FCF_DLGBORDER | FCF_NOBYTEALIGN,
  +                                        fnwpMyDlgProc,
  +                                        "My Dlg Title",
@@ -1773 +1966 @@
  +              WinDestroyWindow(hwndDlg);
  +          }
+ *
+ *      <B>Errors:</B>
+ *
+ *      This does not return a HWND, but an APIRET. This will be
+ *      one of the following:
+ *
+ *      --  NO_ERROR: only in that case, the phwndDlg ptr
+ *          receives the HWND of the new dialog, which can
+ *          then be given to WinProcessDlg. Don't forget
+ *          WinDestroyWindow.
+ *
+ *      --  ERROR_NOT_ENOUGH_MEMORY
+ *
+ *      --  DLGERR_ROW_BEFORE_TABLE: a row definition appeared
+ *          outside a table definition.
+ *
+ *      --  DLGERR_CONTROL_BEFORE_ROW: a control definition
+ *          appeared right after a table definition. You must
+ *          specify a row first.
+ *
+ *      --  DLGERR_NULL_CTL_DEF: TYPE_END_TABLE was specified,
+ *          but the CONTROLDEF ptr was NULL.
+ *
+ *      --  DLGERR_CANNOT_CREATE_FRAME: unable to create the
+ *          WC_FRAME window. Maybe an invalid owner was specified.
+ *
+ *      --  DLGERR_INVALID_CODE: invalid "Type" field in
+ *          DLGHITEM.
+ *
+ *      --  DLGERR_TABLE_NOT_CLOSED, DLGERR_TOO_MANY_TABLES_CLOSED:
+ *          improper nesting of TYPE_START_NEW_TABLE and
+ *          TYPE_END_TABLE fields.
+ *
+ *      --  DLGERR_CANNOT_CREATE_CONTROL: creation of some
+ *          sub-control failed. Maybe an invalid window class
+ *          was specified.
+ *
+ *      --  DLGERR_INVALID_CONTROL_TITLE: bad window title in
+ *          control.
+ *
+ *      --  DLGERR_INVALID_STATIC_BITMAP: static bitmap contains
+ *          an invalid bitmap handle.
  *
  *@@changed V0.9.14 (2001-07-07) [umoeller]: fixed disabled mouse with hwndOwner == HWND_DESKTOP
@@ -1860 +2095 @@
                 HWND    hwndDlg = pDlgData->hwndDlg;
                 HWND    hwndFocusItem = NULLHANDLE;
-                SIZEL   szlClient = {0};
                 RECTL   rclClient;
 
@@ -1868 +2102 @@
                  */
 
-                if (!(arc = Dlg2_CalcSizes(pDlgData,
-                                           &szlClient)))
+                if (!(arc = Dlg2_CalcSizes(pDlgData)))
                 {
                     WinSubclassWindow(hwndDlg, pfnwpDialogProc);
@@ -1881 +2114 @@
                     rclClient.xLeft = 10;
                     rclClient.yBottom = 10;
-                    rclClient.xRight = szlClient.cx + 2 * SPACING;
-                    rclClient.yTop = szlClient.cy + 2 * SPACING;
+                    rclClient.xRight = pDlgData->szlClient.cx + 2 * SPACING;
+                    rclClient.yTop = pDlgData->szlClient.cy + 2 * SPACING;
                     WinCalcFrameRect(hwndDlg,
                                      &rclClient,
@@ -1896 +2129 @@
 
                     arc = Dlg3_PositionAndCreate(pDlgData,
-                                                 &szlClient,
                                                  &hwndFocusItem);
 
@@ -2021 +2253 @@
              */
 
-            Dlg2_CalcSizes(pDlgData,
-                           &szlClient);
+            Dlg2_CalcSizes(pDlgData);
 
             // WinSubclassWindow(hwndDlg, pfnwpDialogProc);
@@ -2054 +2285 @@
             {
                 if (!(arc = Dlg3_PositionAndCreate(pDlgData,
-                                                   &szlClient,
                                                    &hwndFocusItem)))
                     WinSetFocus(HWND_DESKTOP, hwndFocusItem);
@@ -2359 +2589 @@
     ULONG flButtons = flFlags & 0xF;        // low nibble contains MB_YESNO etc.
 
-    const char  *p0 = "Error",
-                *p1 = NULL,
-                *p2 = NULL;
+    PCSZ        p0 = "Error",
+                p1 = NULL,
+                p2 = NULL;
 
     Icon.pcszText = (PCSZ)hptrIcon;
@@ -2672 +2902 @@
                             CTL_COMMON_FONT,
                             0,
-                            { 300, 20 },     // size
+                            { 300, SZL_AUTOSIZE },     // size
                             5               // spacing
                          },