calcMinMaxWidth in AutoTableLayout...

Dirk Mueller mueller@kde.org
Thu, 30 Jan 2003 04:30:07 +0100


On Mit, 29 Jan 2003, David Hyatt wrote:

> This was the cause of the CNN bug I was seeing.  Right before doing the 
> assignment I just added this to fix the bug:

Ok, makes sense. I've committed this: 

--- table_layout.cpp    10 Jan 2003 17:41:57 -0000      1.26
+++ table_layout.cpp    30 Jan 2003 03:29:16 -0000
@@ -554,11 +554,11 @@ void AutoTableLayout::calcMinMaxWidth()
     if ( tw.isFixed() && tw.value > 0 ) {
        minWidth = kMax( minWidth, int( tw.value ) );
        maxWidth = minWidth;
     }
 
-    table->m_maxWidth = maxWidth;
+    table->m_maxWidth = kMin(maxWidth, 32767);
     table->m_minWidth = minWidth;


I'll look into your other propsed table layouting fix now. I think its not 
100% correct, I have to dig out the testcases for that part again..


-- 
Dirk (received 96 mails today)