From fc77b031d74ff6fb38cf10905b4c0e39b567a064 Mon Sep 17 00:00:00 2001
From: John Scipione <jscipione@gmail.com>
Date: Thu, 30 May 2013 21:47:04 -0400
Subject: [PATCH] Revert "BBox: Always offset the top border by the same
 amount"

This reverts commit 39899cf6626bbcb3b41fd44fc49dd2bdec9a34f9.

Checked with BeOS R5, and this is not how it works, to remain
compatable we need to go back to how this was before.
---
 headers/os/interface/Box.h |  2 --
 src/kits/interface/Box.cpp | 45 ++++++++++++++------------------------
 2 files changed, 17 insertions(+), 30 deletions(-)

diff --git a/headers/os/interface/Box.h b/headers/os/interface/Box.h
index 9ab953b9ea..786ea50691 100644
--- a/headers/os/interface/Box.h
+++ b/headers/os/interface/Box.h
@@ -63,7 +63,6 @@ class BBox : public BView {
 		virtual	void		ResizeToPreferred();
 		virtual	void		GetPreferredSize(float* _width, float* _height);
 		virtual	void		MakeFocus(bool focused = true);
-		virtual	void		SetFont(const BFont* font, uint32 mask = B_FONT_ALL);
 		virtual	status_t	GetSupportedSuites(BMessage* message);
 
 		virtual	status_t	Perform(perform_code d, void* arg);
@@ -97,7 +96,6 @@ class BBox : public BView {
 		border_style		fStyle;
 		BView*				fLabelView;
 		LayoutData*			fLayoutData;
-		float				fLabelHeight;
 };
 
 #endif	// _BOX_H
diff --git a/src/kits/interface/Box.cpp b/src/kits/interface/Box.cpp
index 1c03ccddae..e9ff1359f1 100644
--- a/src/kits/interface/Box.cpp
+++ b/src/kits/interface/Box.cpp
@@ -158,7 +158,12 @@ BBox::Border() const
 float
 BBox::TopBorderOffset()
 {
-	return fLabelHeight / 2;
+	_ValidateLayoutData();
+
+	if (fLabel != NULL || fLabelView != NULL)
+		return fLayoutData->label_box.Height() / 2;
+
+	return 0;
 }
 
 
@@ -441,18 +446,6 @@ BBox::MakeFocus(bool focused)
 }
 
 
-void
-BBox::SetFont(const BFont* font, uint32 mask)
-{
-	BView::SetFont(font, mask);
-
-	// recalculate the label height based on the new font
-	font_height fontHeight;
-	GetFontHeight(&fontHeight);
-	fLabelHeight = ceilf(fontHeight.ascent + fontHeight.descent) + 1;
-}
-
-
 status_t
 BBox::GetSupportedSuites(BMessage* message)
 {
@@ -642,11 +635,6 @@ BBox::_InitObject(BMessage* archive)
 
 	if (flags != 0)
 		SetFont(&font, flags);
-	else {
-		font_height fontHeight;
-		GetFontHeight(&fontHeight);
-		fLabelHeight = ceilf(fontHeight.ascent + fontHeight.descent) + 1;
-	}
 
 	if (archive != NULL) {
 		const char* string;
@@ -813,9 +801,8 @@ BBox::_ValidateLayoutData()
 		return;
 
 	// compute the label box, width and height
-	bool hasLabel = true;
-	float labelHeight = 0;
-		// height of the label (pixel count)
+	bool label = true;
+	float labelHeight = 0;	// height of the label (pixel count)
 	if (fLabel) {
 		// leave 6 pixels of the frame, and have a gap of 4 pixels between
 		// the frame and the text on either side
@@ -823,15 +810,14 @@ BBox::_ValidateLayoutData()
 		GetFontHeight(&fontHeight);
 		fLayoutData->label_box.Set(6.0f, 0, 14.0f + StringWidth(fLabel),
 			ceilf(fontHeight.ascent));
-		labelHeight = fLabelHeight;
+		labelHeight = ceilf(fontHeight.ascent + fontHeight.descent) + 1;
 	} else if (fLabelView) {
 		// the label view is placed at (0, 10) at its preferred size
 		BSize size = fLabelView->PreferredSize();
 		fLayoutData->label_box.Set(10, 0, 10 + size.width, size.height);
 		labelHeight = size.height + 1;
 	} else {
-		labelHeight = fLabelHeight;
-		hasLabel = false;
+		label = false;
 	}
 
 	// border
@@ -848,8 +834,8 @@ BBox::_ValidateLayoutData()
 			break;
 	}
 
-	// Grow the top inset by the label height
-	if (labelHeight > fLayoutData->insets.top)
+	// if there's a label, the top inset will be dictated by the label
+	if (label && labelHeight > fLayoutData->insets.top)
 		fLayoutData->insets.top = labelHeight;
 
 	// total number of pixel the border adds
@@ -857,8 +843,11 @@ BBox::_ValidateLayoutData()
 	float addHeight = fLayoutData->insets.top + fLayoutData->insets.bottom;
 
 	// compute the minimal width induced by the label
-	float minWidth = !hasLabel ? addWidth - 1
-		: fLayoutData->label_box.right + fLayoutData->insets.right;
+	float minWidth;
+	if (label)
+		minWidth = fLayoutData->label_box.right + fLayoutData->insets.right;
+	else
+		minWidth = addWidth - 1;
 
 	// finally consider the child constraints, if we shall support layout
 	BView* child = _Child();