From 2659cdba83dc62cbff17e5dc4f9691ccff8df8da Mon Sep 17 00:00:00 2001 From: John Scipione Date: Fri, 15 Nov 2019 20:07:31 -0500 Subject: [PATCH] ScreenSaver prefs: Invoke list item runs test This copies the behavior of BeOS R5 ScreenSaver prefs. Don't invoke the listview on AllAttached() anymore, this would make the selected screensaver start when the pref window opens. Horizontally center the Test button. Change-Id: I65471112d37d9efd17945f05bce485ff2fece9dd Reviewed-on: https://review.haiku-os.org/c/haiku/+/1958 Reviewed-by: Adrien Destugues --- src/preferences/screensaver/ScreenSaverWindow.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/preferences/screensaver/ScreenSaverWindow.cpp b/src/preferences/screensaver/ScreenSaverWindow.cpp index 63bed647d7..5663e16a82 100644 --- a/src/preferences/screensaver/ScreenSaverWindow.cpp +++ b/src/preferences/screensaver/ScreenSaverWindow.cpp @@ -577,6 +577,8 @@ ModulesView::ModulesView(const char* name, ScreenSaverSettings& settings) fScreenSaversListView->SetSelectionMessage( new BMessage(kMsgSaverSelected)); + fScreenSaversListView->SetInvocationMessage( + new BMessage(kMsgTestSaver)); BScrollView* saversListScrollView = new BScrollView("scroll_list", fScreenSaversListView, 0, false, true); @@ -592,10 +594,7 @@ ModulesView::ModulesView(const char* name, ScreenSaverSettings& settings) .AddGroup(B_VERTICAL) .Add(fPreviewView) .Add(saversListScrollView) - .AddGroup(B_HORIZONTAL) - .Add(fTestButton) - .AddGlue() - .End() + .Add(fTestButton) .End() .Add(fSettingsBox) .End(); @@ -634,7 +633,6 @@ void ModulesView::AllAttached() { PopulateScreenSaverList(); - fScreenSaversListView->Invoke(); }