From 6aea66ee0dfbc7de0680b520d17a82efe43116ed Mon Sep 17 00:00:00 2001 From: John Scipione Date: Sun, 2 Jun 2013 17:12:42 -0400 Subject: [PATCH] Backgrounds: Refactor view layout code * Eliminate an uneeded group level, top level is B_VERTICAL * Remove B_USE_DEFAULT_SPACING calls, use default param instead --- .../backgrounds/BackgroundsView.cpp | 20 +++++++++---------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/src/preferences/backgrounds/BackgroundsView.cpp b/src/preferences/backgrounds/BackgroundsView.cpp index f57f84059a..7718482664 100644 --- a/src/preferences/backgrounds/BackgroundsView.cpp +++ b/src/preferences/backgrounds/BackgroundsView.cpp @@ -209,18 +209,16 @@ BackgroundsView::BackgroundsView() fApply->SetExplicitAlignment(BAlignment(B_ALIGN_RIGHT, B_ALIGN_NO_VERTICAL)); - AddChild(BLayoutBuilder::Group<>() - .AddGroup(B_VERTICAL, B_USE_DEFAULT_SPACING) - .AddGroup(B_HORIZONTAL, B_USE_DEFAULT_SPACING) - .Add(previewBox) - .Add(rightbox) - .End() - .AddGroup(B_HORIZONTAL, B_USE_DEFAULT_SPACING) - .Add(fRevert) - .Add(fApply) - .End() - .SetInsets(B_USE_DEFAULT_SPACING) + AddChild(BLayoutBuilder::Group<>(B_VERTICAL) + .AddGroup(B_HORIZONTAL) + .Add(previewBox) + .Add(rightbox) .End() + .AddGroup(B_HORIZONTAL) + .Add(fRevert) + .Add(fApply) + .End() + .SetInsets(B_USE_DEFAULT_SPACING) .View()); fApply->MakeDefault(true);