From 263e936e87a82b9174cc10bbbe1e43010cb92946 Mon Sep 17 00:00:00 2001
From: John Scipione <jscipione@gmail.com>
Date: Wed, 13 Apr 2016 10:37:56 -0700
Subject: [PATCH] Gravity: Fixup scroll bar range

---
 src/add-ons/screen_savers/gravity/ConfigView.cpp | 15 +++++++++++++--
 src/add-ons/screen_savers/gravity/ConfigView.h   |  1 +
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/src/add-ons/screen_savers/gravity/ConfigView.cpp b/src/add-ons/screen_savers/gravity/ConfigView.cpp
index 835b4b0cf1..b0fe21ed93 100644
--- a/src/add-ons/screen_savers/gravity/ConfigView.cpp
+++ b/src/add-ons/screen_savers/gravity/ConfigView.cpp
@@ -53,8 +53,6 @@ ConfigView::ConfigView(BRect frame, Gravity* parent)
 	fShadeList->AddItem(new ColorItem("White", (rgb_color){ 255, 255, 255 }));
 	fShadeList->AddItem(new RainbowItem("Rainbow"));
 
-	fShadeList->Select(parent->Config.ShadeID);
-
 	fShadeScroll = new BScrollView(B_EMPTY_STRING, fShadeList,
 		B_WILL_DRAW | B_FRAME_EVENTS, false, true);
 
@@ -76,6 +74,19 @@ ConfigView::ConfigView(BRect frame, Gravity* parent)
 }
 
 
+void
+ConfigView::AllAttached()
+{
+	// fixup scroll bar range
+	fShadeScroll->ScrollBar(B_VERTICAL)->SetRange(0.0f,
+		fShadeList->ItemFrame(fShadeList->CountItems()).bottom
+			- fShadeList->ItemFrame((int32)0).top);
+
+	// select the shade
+	fShadeList->Select(fParent->Config.ShadeID);
+}
+
+
 void
 ConfigView::AttachedToWindow()
 {
diff --git a/src/add-ons/screen_savers/gravity/ConfigView.h b/src/add-ons/screen_savers/gravity/ConfigView.h
index 307ec46288..3cc596c239 100644
--- a/src/add-ons/screen_savers/gravity/ConfigView.h
+++ b/src/add-ons/screen_savers/gravity/ConfigView.h
@@ -27,6 +27,7 @@ class ConfigView : public BView {
 public:
 								ConfigView(BRect frame, Gravity* parent);
 
+			void				AllAttached();
 			void				AttachedToWindow();
 			void				MessageReceived(BMessage* message);