From dd822ba860f959304953d0a6c319c2b29f85dd13 Mon Sep 17 00:00:00 2001
From: John Scipione <jscipione@gmail.com>
Date: Wed, 8 Jul 2015 12:09:02 -0700
Subject: [PATCH] Nebula: Style screen saver prefs

* Make the title the default font and bold like other screen savers
* Don't center the copyright text, left align
* Put spacing strut between title/copyright and the rest
* Put the pop-up menus in a grid, make them variable width
* Add some more glue so items grow apart more nicely
---
 src/add-ons/screen_savers/nebula/Nebula.cpp | 33 +++++++++++++++++----
 1 file changed, 27 insertions(+), 6 deletions(-)

diff --git a/src/add-ons/screen_savers/nebula/Nebula.cpp b/src/add-ons/screen_savers/nebula/Nebula.cpp
index 086e5266f5..b929ffd3b2 100644
--- a/src/add-ons/screen_savers/nebula/Nebula.cpp
+++ b/src/add-ons/screen_savers/nebula/Nebula.cpp
@@ -7,6 +7,7 @@
  */
 
 #include <AppKit.h>
+#include <ControlLook.h>
 #include <InterfaceKit.h>
 #include <LayoutBuilder.h>
 #include <Window.h>
@@ -443,11 +444,10 @@ SettingsView::AttachedToWindow()
 	SetViewColor(ui_color(B_PANEL_BACKGROUND_COLOR));
 
 	BStringView* titleString = new BStringView(B_EMPTY_STRING, "Nebula");
-	titleString->SetFontSize(18.0);
+	titleString->SetFont(be_bold_font);
 
 	BStringView* copyrightString = new BStringView(B_EMPTY_STRING,
 		"© 2001-2004 Axel Dörfler.");
-	copyrightString->SetAlignment(B_ALIGN_CENTER);
 
 	BPopUpMenu *popMenu = new BPopUpMenu("");
 	BMenuItem *item;
@@ -515,11 +515,32 @@ SettingsView::AttachedToWindow()
 			B_USE_BIG_INSETS, B_USE_HALF_ITEM_INSETS)
 		.Add(titleString)
 		.Add(copyrightString)
-		.Add(fColorMenu)
-		.Add(fWidthMenu)
-		.Add(fBorderMenu)
-		.Add(fMotionCheck)
+		.AddStrut(roundf(be_control_look->DefaultItemSpacing() / 2))
+		.AddGlue()
+		.AddGroup(B_HORIZONTAL, 0.0f)
+			.AddGrid(B_USE_DEFAULT_SPACING, B_USE_SMALL_SPACING)
+				.Add(fColorMenu->CreateLabelLayoutItem(), 0, 0)
+				.AddGroup(B_HORIZONTAL, 0.0f, 1, 0)
+					.Add(fColorMenu->CreateMenuBarLayoutItem(), 0.0f)
+					.AddGlue()
+					.End()
+				.Add(fWidthMenu->CreateLabelLayoutItem(), 0, 1)
+				.AddGroup(B_HORIZONTAL, 0.0f, 1, 1)
+					.Add(fWidthMenu->CreateMenuBarLayoutItem(), 0.0f)
+					.AddGlue()
+					.End()
+				.Add(fBorderMenu->CreateLabelLayoutItem(), 0, 2)
+				.AddGroup(B_HORIZONTAL, 0.0f, 1, 2)
+					.Add(fBorderMenu->CreateMenuBarLayoutItem(), 0.0f)
+					.AddGlue()
+					.End()
+				.Add(fMotionCheck, 1, 3)
+				.End()
+			.AddGlue()
+			.End()
+		.AddGlue()
 		.Add(fSpeedSlider)
+		.AddGlue()
 		.Add(fFramesSlider)
 		.AddGlue()
 	.End();