Clamp width to min/max in getAvailWidthOfChild()

When the width is set to auto, a different branch is used to compute the
width via getAvailWidth(true). The returned witdh needs to be clamped to
be in the min/max-width CSS range if given, as otherwise it won't be
corrected.

Fixes: https://github.com/dillo-browser/dillo/issues/89

1 files changed, 19 insertions, 3 deletions

diff --git a/dw/textblock.cc b/dw/textblock.cc
index 27438b6d..0cef739b 100644
--- a/dw/textblock.cc
+++ b/dw/textblock.cc
@@ -2,7 +2,7 @@
  * Dillo Widget
  *
  * Copyright 2005-2007, 2012-2014 Sebastian Geerken <sgeerken@dillo.org>
- * Copyright 2023 Rodrigo Arias Mallo <rodarima@gmail.com>
+ * Copyright 2023-2024 Rodrigo Arias Mallo <rodarima@gmail.com>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -768,12 +768,28 @@ int Textblock::getAvailWidthOfChild (Widget *child, bool forceValue)
          // No width specified: similar to standard implementation (see
          // there), but "leftInnerPadding" has to be considered, too.
          DBG_OBJ_MSG ("resize", 1, "no specification");
-         if (forceValue)
+         if (forceValue) {
             width = misc::max (getAvailWidth (true) - boxDiffWidth ()
                                - leftInnerPadding,
                                0);
-         else
+
+            if (width != -1) {
+               /* Clamp to min-width and max-width if given, taking into
+                * account leftInnerPadding. */
+               int maxWidth = child->calcWidth (child->getStyle()->maxWidth,
+                     -1, this, -1, false);
+               if (maxWidth != -1 && width > maxWidth - leftInnerPadding)
+                  width = maxWidth - leftInnerPadding;
+
+               int minWidth = child->calcWidth (child->getStyle()->minWidth,
+                     -1, this, -1, false);
+               if (minWidth != -1 && width < minWidth - leftInnerPadding)
+                  width = minWidth - leftInnerPadding;
+            }
+
+         } else {
             width = -1;
+         }
       } else
          width = Widget::getAvailWidthOfChild (child, forceValue);