From 4d2baf30dfbd8154b94d7b6dedd9073ebfa48fee Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Stephan=20A=C3=9Fmus?= <superstippi@gmx.de>
Date: Tue, 15 Jul 2008 13:23:29 +0000
Subject: [PATCH] * Small coding style improvements. * fPreferredSize was not
 initialized for the archive constructor. * Added comment to archive
 constructor, because I was wondering how   the default button status was
 reconstructed or the archive code path.

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@26424 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
 src/kits/interface/Button.cpp | 28 +++++++++++++++++++---------
 1 file changed, 19 insertions(+), 9 deletions(-)

diff --git a/src/kits/interface/Button.cpp b/src/kits/interface/Button.cpp
index 2ce9f539b7..c277ff4e30 100644
--- a/src/kits/interface/Button.cpp
+++ b/src/kits/interface/Button.cpp
@@ -60,10 +60,13 @@ BButton::~BButton()
 
 
 BButton::BButton(BMessage *archive)
-	:	BControl (archive)
+	:	BControl(archive),
+		fPreferredSize(-1, -1)
 {
 	if (archive->FindBool("_default", &fDrawAsDefault) != B_OK)
 		fDrawAsDefault = false;
+	// NOTE: Default button state will be synchronized with the window
+	// in AttachedToWindow().
 }
 
 
@@ -247,7 +250,7 @@ BButton::Draw(BRect updateRect)
 	const float x = (rect.right - stringWidth) / 2.0;
 	const float labelY = bounds.top
 		+ ((bounds.Height() - fh.ascent - fh.descent) / 2.0)
-		+ fh.ascent +  1.0;
+		+ fh.ascent + 1.0;
 	const float focusLineY = labelY + fh.descent;
 
 	/* speed trick:
@@ -260,8 +263,10 @@ BButton::Draw(BRect updateRect)
 		if (pushed) {
 			rect.InsetBy(2.0, 2.0);
 			InvertRect(rect);
-		} else 
-			DrawFocusLine(x, focusLineY, stringWidth, IsFocus() && Window()->IsActive());
+		} else {
+			DrawFocusLine(x, focusLineY, stringWidth, IsFocus()
+				&& Window()->IsActive());
+		}
 		
 		return;
 	}
@@ -294,19 +299,24 @@ BButton::Draw(BRect updateRect)
 		bevelColor2 = panelBgColor;
 
 		if (IsDefault()) {
-			borderBevelShadow = tint_color(dark1BorderColor, (B_NO_TINT + B_DARKEN_1_TINT) / 2);
+			borderBevelShadow = tint_color(dark1BorderColor,
+				(B_NO_TINT + B_DARKEN_1_TINT) / 2);
 			borderBevelLight = tint_color(dark1BorderColor, B_LIGHTEN_1_TINT);
 
-			borderBevelLight.red = (borderBevelLight.red + panelBgColor.red) / 2;
-			borderBevelLight.green = (borderBevelLight.green + panelBgColor.green) / 2;
-			borderBevelLight.blue = (borderBevelLight.blue + panelBgColor.blue) / 2;
+			borderBevelLight.red = (borderBevelLight.red + panelBgColor.red)
+				/ 2;
+			borderBevelLight.green = (borderBevelLight.green
+				+ panelBgColor.green) / 2;
+			borderBevelLight.blue = (borderBevelLight.blue
+				+ panelBgColor.blue) / 2;
 	
 			dark1BorderColor = tint_color(dark1BorderColor, B_DARKEN_3_TINT);
 			dark2BorderColor = tint_color(dark1BorderColor, B_DARKEN_4_TINT);
 
 			bevelColorRBCorner = borderBevelShadow;
 		} else {
-			borderBevelShadow = tint_color(panelBgColor, (B_NO_TINT + B_DARKEN_1_TINT) / 2);
+			borderBevelShadow = tint_color(panelBgColor,
+				(B_NO_TINT + B_DARKEN_1_TINT) / 2);
 			borderBevelLight = buttonBgColor;
 
 			bevelColorRBCorner = dark1BorderColor;