[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 80341] Kicker applet's width includes grip width
From:       John Firebaugh <jfirebaugh () kde ! org>
Date:       2004-05-24 0:58:32
Message-ID: 20040524005832.26347.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
      
http://bugs.kde.org/show_bug.cgi?id=80341      
jfirebaugh kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From jfirebaugh kde org  2004-05-24 02:58 -------
CVS commit by firebaugh: 

Don't add space for handles when they're hidden.

CCMAIL:80341-done bugs kde org


  M +12 -4     container_applet.cpp   1.59


--- kdebase/kicker/core/container_applet.cpp  #1.58:1.59
 @ -313,5 +313,7  @ int InternalAppletContainer::widthForHei
             return h + _handle->widthForHeight(h);
     }
-    return _applet->widthForHeight(h) + _handle->widthForHeight(h);
+    return _applet->widthForHeight(h) + ( _handle->isVisible()
+                                        ? _handle->widthForHeight(h)
+                                        : 0 );
 }
 
 @ -324,5 +326,7  @ int InternalAppletContainer::heightForWi
             return w + _handle->heightForWidth(w);
     }
-    return _applet->heightForWidth(w) + _handle->heightForWidth(w);
+    return _applet->heightForWidth(w) + ( _handle->isVisible()
+                                        ? _handle->heightForWidth(w)
+                                        : 0 );
 }
 
 @ -511,5 +515,7  @ int ExternalAppletContainer::widthForHei
         reply >> w;
     }
-    return w + _handle->widthForHeight(h);
+    return w + ( _handle->isVisible()
+               ? _handle->widthForHeight(h)
+               : 0 );
 }
 
 @ -535,5 +541,7  @ int ExternalAppletContainer::heightForWi
         reply >> h;
     }
-    return h + _handle->heightForWidth(w);
+    return h + ( _handle->isVisible()
+               ? _handle->heightForWidth(w)
+               : 0 );
 }
[prev in list] [next in list] [prev in thread] [next in thread]