Mail: corrected spacing after window spacing change.

* The button panel shouldn't be indented at all. * The header panel was intented twice, once by the surrounding layout, once by its own.
haiku · Nov 16, 2015 · 28da8a4 · 28da8a4
1 parent 3c9d8d6
commit 28da8a4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/apps/mail/Header.cpp b/src/apps/mail/Header.cpp
@@ -402,7 +402,6 @@ THeaderView::THeaderView(bool incoming, bool resending, int32 defaultAccount)
 
 	BGridLayout* layout = GridLayout();
 
-	layout->SetInsets(B_USE_DEFAULT_SPACING);
 	if (fIncoming)
 		layout->SetHorizontalSpacing(0);
 	layout->SetVerticalSpacing(B_USE_HALF_ITEM_SPACING);

diff --git a/src/apps/mail/MailWindow.cpp b/src/apps/mail/MailWindow.cpp
@@ -557,11 +557,11 @@ TMailWindow::TMailWindow(BRect rect, const char* title, TMailApp* app,
 
 	BLayoutBuilder::Group<>(this, B_VERTICAL, 0)
 		.Add(fMenuBar)
+		.Add(fToolBar)
 		.AddGroup(B_VERTICAL, 0)
-			.Add(fToolBar)
 			.Add(fHeaderView)
-			.SetInsets(B_USE_WINDOW_SPACING, 0, B_USE_WINDOW_SPACING, 0)
-			.End()
+			.SetInsets(B_USE_WINDOW_SPACING, B_USE_DEFAULT_SPACING)
+		.End()
 		.Add(fContentView);
 
 	if (to != NULL)