[haiku-commits] r38064 - in haiku/trunk/src: add-ons/disk_systems/bfs add-ons/disk_systems/intel apps/deskbar apps/devices apps/launchbox ...

  • From: yourpalal2@xxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 12 Aug 2010 18:54:04 +0200 (CEST)

Author: yourpalal
Date: 2010-08-12 18:54:03 +0200 (Thu, 12 Aug 2010)
New Revision: 38064
Changeset: http://dev.haiku-os.org/changeset/38064
Ticket: http://dev.haiku-os.org/ticket/6407

Modified:
   haiku/trunk/src/add-ons/disk_systems/bfs/InitializeParameterEditor.cpp
   haiku/trunk/src/add-ons/disk_systems/intel/CreationParameterEditor.cpp
   haiku/trunk/src/apps/deskbar/CalendarMenuWindow.cpp
   haiku/trunk/src/apps/devices/DevicesView.cpp
   haiku/trunk/src/apps/launchbox/NamePanel.cpp
   haiku/trunk/src/apps/packageinstaller/PackageStatus.cpp
   haiku/trunk/src/apps/packageinstaller/PackageStatus.h
   haiku/trunk/src/apps/terminal/AppearPrefView.cpp
   haiku/trunk/src/apps/terminal/FindWindow.cpp
   haiku/trunk/src/preferences/filetypes/ApplicationTypeWindow.cpp
   haiku/trunk/src/preferences/filetypes/ApplicationTypesWindow.cpp
   haiku/trunk/src/preferences/filetypes/AttributeWindow.cpp
   haiku/trunk/src/preferences/filetypes/ExtensionWindow.cpp
   haiku/trunk/src/preferences/filetypes/FileTypeWindow.cpp
   haiku/trunk/src/preferences/filetypes/FileTypesWindow.cpp
   haiku/trunk/src/preferences/fonts/FontSelectionView.cpp
   haiku/trunk/src/preferences/fonts/MainWindow.cpp
   haiku/trunk/src/preferences/locale/FormatSettingsView.cpp
   haiku/trunk/src/preferences/screen/ScreenWindow.cpp
Log:
Update some apps/add-ons/preflets for impending changes to the layout API, 
specifically the layout builders. Also do a bit of style and layout cleanup in 
some places. This is part of #6407. Everything I've updated should look and 
function the same as it did before.


Modified: haiku/trunk/src/add-ons/disk_systems/bfs/InitializeParameterEditor.cpp
===================================================================
--- haiku/trunk/src/add-ons/disk_systems/bfs/InitializeParameterEditor.cpp      
2010-08-12 14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/add-ons/disk_systems/bfs/InitializeParameterEditor.cpp      
2010-08-12 16:54:03 UTC (rev 38064)
@@ -140,6 +140,6 @@
                .Add(fBlockSizeMF->CreateMenuBarLayoutItem(), 1, 1)
 
                // row 3
-               .Add(fUseIndicesCB, 0, 2, 2)
+               .Add(fUseIndicesCB, 0, 2, 2).View()
        ;
 }

Modified: haiku/trunk/src/add-ons/disk_systems/intel/CreationParameterEditor.cpp
===================================================================
--- haiku/trunk/src/add-ons/disk_systems/intel/CreationParameterEditor.cpp      
2010-08-12 14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/add-ons/disk_systems/intel/CreationParameterEditor.cpp      
2010-08-12 16:54:03 UTC (rev 38064)
@@ -7,7 +7,7 @@
 #include "CreationParameterEditor.h"
 
 #include <DiskDeviceTypes.h>
-#include <GroupLayoutBuilder.h>
+#include <GroupView.h>
 #include <PartitionParameterEditor.h>
 #include <View.h>
 
@@ -20,9 +20,8 @@
        fParameters(NULL)
 {
        fActiveCB = new BCheckBox("active", "Active partition", NULL);
-       fView = BGroupLayoutBuilder(B_VERTICAL)
-               .Add(fActiveCB)
-       ;
+       fView = new BGroupView(B_VERTICAL);
+       fView->AddChild(fActiveCB);
 }
 
 

Modified: haiku/trunk/src/apps/deskbar/CalendarMenuWindow.cpp
===================================================================
--- haiku/trunk/src/apps/deskbar/CalendarMenuWindow.cpp 2010-08-12 14:50:57 UTC 
(rev 38063)
+++ haiku/trunk/src/apps/deskbar/CalendarMenuWindow.cpp 2010-08-12 16:54:03 UTC 
(rev 38064)
@@ -129,7 +129,8 @@
        BView* view = BGroupLayoutBuilder(B_VERTICAL, 5.0)
                .Add(gridLayout->View())
                .Add(fCalendarView)
-               .SetInsets(5.0, 5.0, 5.0, 5.0);
+               .SetInsets(5.0, 5.0, 5.0, 5.0)
+               .TopView();
        groupView->AddChild(view);
        AddChild(groupView);
 

Modified: haiku/trunk/src/apps/devices/DevicesView.cpp
===================================================================
--- haiku/trunk/src/apps/devices/DevicesView.cpp        2010-08-12 14:50:57 UTC 
(rev 38063)
+++ haiku/trunk/src/apps/devices/DevicesView.cpp        2010-08-12 16:54:03 UTC 
(rev 38064)
@@ -88,16 +88,17 @@
        fTabView->AddTab(fAttributesView, fDetailedTab);
        fDetailedTab->SetLabel("Detailed");
 
-       AddChild(BGroupLayoutBuilder(B_VERTICAL,0)
+       AddChild(BGroupLayoutBuilder(B_VERTICAL, 0)
                                .Add(menuBar)
                                .Add(BSplitLayoutBuilder(B_HORIZONTAL, 5)
                                        .Add(BGroupLayoutBuilder(B_VERTICAL, 5)
                                                .Add(fOrderByMenu, 1)
                                                .Add(scrollView, 2)
+                                               .TopView()
                                        )
                                        .Add(fTabView, 2)
                                        .SetInsets(5, 5, 5, 5)
-                               )
+                               ).TopView()
                        );
 }
 

Modified: haiku/trunk/src/apps/launchbox/NamePanel.cpp
===================================================================
--- haiku/trunk/src/apps/launchbox/NamePanel.cpp        2010-08-12 14:50:57 UTC 
(rev 38063)
+++ haiku/trunk/src/apps/launchbox/NamePanel.cpp        2010-08-12 16:54:03 UTC 
(rev 38064)
@@ -6,8 +6,7 @@
 #include <stdio.h>
 
 #include <Button.h>
-#include <GroupLayoutBuilder.h>
-#include <SpaceLayoutItem.h>
+#include <LayoutBuilder.h>
 #include <Screen.h>
 #include <TextControl.h>
 
@@ -35,36 +34,31 @@
                new BMessage(MSG_PANEL_CANCEL));
        fNameTC = new BTextControl(label, text, NULL);
 
-       BView* topView = BGroupLayoutBuilder(B_VERTICAL, 10)
+       BLayoutBuilder::Group<>(this, B_VERTICAL, 10)
                .AddGlue()
 
                // controls
-               .Add(BGroupLayoutBuilder(B_HORIZONTAL, 5)
-                       .Add(BSpaceLayoutItem::CreateHorizontalStrut(5))
+               .AddGroup(B_HORIZONTAL, 5)
+                       .AddStrut(5)
 
                        // text control
                        .Add(fNameTC->CreateLabelLayoutItem())
                        .Add(fNameTC->CreateTextViewLayoutItem())
+                       .AddStrut(5)
+                       .End()
 
-                       .Add(BSpaceLayoutItem::CreateHorizontalStrut(5))
-               )
-
                .AddGlue()
 
                // buttons
-               .Add(BGroupLayoutBuilder(B_HORIZONTAL, 5)
-                       .Add(BSpaceLayoutItem::CreateGlue())
+               .AddGroup(B_HORIZONTAL, 5)
+                       .AddGlue()
                        .Add(cancelButton)
                        .Add(defaultButton)
-                       .Add(BSpaceLayoutItem::CreateHorizontalStrut(5))
-               )
+                       .AddStrut(5)
+                       .End()
 
-               .AddGlue()
-       ;
+               .AddGlue();
 
-       SetLayout(new BGroupLayout(B_HORIZONTAL));      
-       AddChild(topView);
-
        SetDefaultButton(defaultButton);
        fNameTC->MakeFocus(true);
 

Modified: haiku/trunk/src/apps/packageinstaller/PackageStatus.cpp
===================================================================
--- haiku/trunk/src/apps/packageinstaller/PackageStatus.cpp     2010-08-12 
14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/apps/packageinstaller/PackageStatus.cpp     2010-08-12 
16:54:03 UTC (rev 38064)
@@ -11,8 +11,8 @@
 
 #include <Autolock.h>
 #include <Catalog.h>
-#include <GroupLayoutBuilder.h>
 #include <GroupLayout.h>
+#include <LayoutBuilder.h>
 #include <Locale.h>
 
 #include <stdio.h>
@@ -43,57 +43,9 @@
 }
 
 
-
-
-
 // #pragma mark -
 
 
-/*PackageStatus::PackageStatus(BHandler *parent, const char *title,
-               const char *label, const char *trailing)
-       :       BWindow(BRect(200, 200, 550, 275), title, B_TITLED_WINDOW,
-                       B_NOT_CLOSABLE | B_NOT_RESIZABLE | B_NOT_ZOOMABLE, 0)
-{
-       SetSizeLimits(0, 100000, 0, 100000);
-       fBackground = new BView(Bounds(), "background", B_FOLLOW_NONE, 
B_WILL_DRAW);
-       fBackground->SetViewColor(ui_color(B_PANEL_BACKGROUND_COLOR));
-
-       BRect rect(Bounds());
-       float width, height;
-       rect.left += 6;
-       rect.right -= 40;
-       rect.top += 6;
-       rect.bottom = rect.top + 15;
-       fStatus = new BStatusBar(rect, "status_bar", T("Installing package"));
-       fStatus->SetBarHeight(12);
-       fStatus->GetPreferredSize(&width, &height);
-       fStatus->ResizeTo(fStatus->Frame().Width(), height);
-       fBackground->AddChild(fStatus);
-
-       font_height fontHeight;
-       fBackground->GetFontHeight(&fontHeight);
-       BRect frame = fStatus->Frame();
-       fBackground->ResizeTo(Bounds().Width(), (2 * frame.top) + 
frame.Height() +
-                       fontHeight.leading + fontHeight.ascent + 
fontHeight.descent);
-
-       rect = Bounds();
-       rect.left = rect.right - 32;
-       //rect.right = rect.left + 17;
-       rect.top += 18;
-       //rect.bottom = rect.top + 10;
-       fButton = new StopButton();
-       fButton->MoveTo(BPoint(rect.left, rect.top));
-       fButton->ResizeTo(22, 18);
-       fBackground->AddChild(fButton);
-
-       AddChild(fBackground);
-       fButton->SetTarget(parent);
-
-       ResizeTo(Bounds().Width(), fBackground->Bounds().Height());
-       Run();
-}*/
-
-
 PackageStatus::PackageStatus(const char *title, const char *label,
                const char *trailing, BHandler *parent)
        :
@@ -102,22 +54,17 @@
        fIsStopped(false),
        fParent(parent)
 {
-       SetLayout(new BGroupLayout(B_VERTICAL));
-
        fStatus = new BStatusBar("status_bar", B_TRANSLATE("Installing 
package"));
        fStatus->SetBarHeight(12);
 
        fButton = new StopButton();
        fButton->SetExplicitMaxSize(BSize(22, 18));
 
-       fBackground = BGroupLayoutBuilder(B_HORIZONTAL)
+       BLayoutBuilder::Group<>(this, B_HORIZONTAL, 0)
                .AddStrut(5.0f)
                .Add(fStatus)
                .Add(fButton);
-       fBackground->SetViewColor(ui_color(B_PANEL_BACKGROUND_COLOR));
 
-       AddChild(fBackground);
-
        fButton->SetTarget(this);
        Run();
 }

Modified: haiku/trunk/src/apps/packageinstaller/PackageStatus.h
===================================================================
--- haiku/trunk/src/apps/packageinstaller/PackageStatus.h       2010-08-12 
14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/apps/packageinstaller/PackageStatus.h       2010-08-12 
16:54:03 UTC (rev 38064)
@@ -42,7 +42,6 @@
                bool Stopped();
                
        private:
-               BView *fBackground;
                BStatusBar *fStatus;
                StopButton *fButton;
                bool fIsStopped;

Modified: haiku/trunk/src/apps/terminal/AppearPrefView.cpp
===================================================================
--- haiku/trunk/src/apps/terminal/AppearPrefView.cpp    2010-08-12 14:50:57 UTC 
(rev 38063)
+++ haiku/trunk/src/apps/terminal/AppearPrefView.cpp    2010-08-12 16:54:03 UTC 
(rev 38064)
@@ -15,8 +15,6 @@
 #include <Catalog.h>
 #include <CheckBox.h>
 #include <ColorControl.h>
-#include <GridLayoutBuilder.h>
-#include <GroupLayoutBuilder.h>
 #include <LayoutBuilder.h>
 #include <Locale.h>
 #include <Menu.h>
@@ -84,8 +82,6 @@
                NULL
        };
 
-       SetLayout(new BGroupLayout(B_HORIZONTAL));
-
        fWarnOnExit = new BCheckBox(
                B_TRANSLATE("Confirm exit if active programs exist"),
                        new BMessage(MSG_WARN_ON_EXIT_CHANGED));
@@ -100,8 +96,8 @@
        fFont = new BMenuField(B_TRANSLATE("Font:"), fontMenu);
        fFontSize = new BMenuField(B_TRANSLATE("Size:"), sizeMenu);
 
-       BPopUpMenu* schemasPopUp 
=_MakeColorSchemaMenu(MSG_COLOR_SCHEMA_CHANGED, gPredefinedSchemas,
-               gPredefinedSchemas[0]);
+       BPopUpMenu* schemasPopUp =_MakeColorSchemaMenu(MSG_COLOR_SCHEMA_CHANGED,
+               gPredefinedSchemas, gPredefinedSchemas[0]);
        fColorSchemaField = new BMenuField(B_TRANSLATE("Color schema:"),
                schemasPopUp);
 
@@ -112,10 +108,12 @@
                        colorsPopUp);
        fColorField->SetEnabled(false);
 
-       BView* layoutView = BLayoutBuilder::Group<>()
+       BGroupLayout* layout = new BGroupLayout(B_HORIZONTAL);
+       SetLayout(layout);
+       BLayoutBuilder::Group<>(layout)
                .SetInsets(5, 5, 5, 5)
                .AddGroup(B_VERTICAL, 5)
-                       .Add(BGridLayoutBuilder(5, 5)
+                       .AddGrid(5, 5)
                                .Add(fFont->CreateLabelLayoutItem(), 0, 0)
                                .Add(fFont->CreateMenuBarLayoutItem(), 1, 0)
                                .Add(fFontSize->CreateLabelLayoutItem(), 0, 1)
@@ -124,17 +122,13 @@
                                
.Add(fColorSchemaField->CreateMenuBarLayoutItem(), 1, 2)
                                .Add(fColorField->CreateLabelLayoutItem(), 0, 3)
                                .Add(fColorField->CreateMenuBarLayoutItem(), 1, 
3)
-                               )
+                       .       End()
                        .AddGroup(B_VERTICAL, 5)
                                .AddGlue()
                                .Add(fColorControl = new 
BColorControl(BPoint(10, 10),
                                        B_CELLS_32x8, 8.0, "", new 
BMessage(MSG_COLOR_CHANGED)))
-                               .Add(fWarnOnExit)
-                       .End()
-               .End();
+                               .Add(fWarnOnExit);
 
-       AddChild(layoutView);
-
        fFont->SetAlignment(B_ALIGN_RIGHT);
        fFontSize->SetAlignment(B_ALIGN_RIGHT);
        fColorField->SetAlignment(B_ALIGN_RIGHT);
@@ -421,7 +415,8 @@
                        if (sizes[i] > defaultSize) {
                                BString string;
                                string << defaultSize;
-                               BMenuItem* item = new 
BMenuItem(string.String(), new BMessage(command));
+                               BMenuItem* item = new BMenuItem(string.String(),
+                                       new BMessage(command));
                                item->SetMarked(true);
                                menu->AddItem(item, i);
                                break;

Modified: haiku/trunk/src/apps/terminal/FindWindow.cpp
===================================================================
--- haiku/trunk/src/apps/terminal/FindWindow.cpp        2010-08-12 14:50:57 UTC 
(rev 38063)
+++ haiku/trunk/src/apps/terminal/FindWindow.cpp        2010-08-12 16:54:03 UTC 
(rev 38064)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2007-2009, Haiku, Inc. All rights reserved.
+ * Copyright 2007-2010, Haiku, Inc. All rights reserved.
  * Copyright 2003-2004 Kian Duffy, myob@xxxxxxxxxxxxxxxxxxxxx
  * Parts Copyright 1998-1999 Kazuho Okui and Takashi Murai.
  * All rights reserved. Distributed under the terms of the MIT license.
@@ -45,7 +45,7 @@
 
        BRadioButton* useSelection = NULL;
        const float spacing = be_control_look->DefaultItemSpacing();
-       BView* layoutView = BGroupLayoutBuilder(B_VERTICAL, 5.0)
+       AddChild(BGroupLayoutBuilder(B_VERTICAL, 5.0)
                .SetInsets(spacing, spacing, spacing, spacing)
                .Add(BGridLayoutBuilder()
                        .Add(fTextRadio = new BRadioButton(B_TRANSLATE("Use 
text:"),
@@ -57,12 +57,10 @@
                .Add(fForwardSearchBox = new BCheckBox(B_TRANSLATE("Search 
forward")))
                .Add(fMatchCaseBox = new BCheckBox(B_TRANSLATE("Match case")))
                .Add(fMatchWordBox = new BCheckBox(B_TRANSLATE("Match word")))
-               .Add(fFindButton = new BButton(B_TRANSLATE("Find"), new 
BMessage(MSG_FIND)))
-               .End();
-       AddChild(layoutView);
+               .Add(fFindButton = new BButton(B_TRANSLATE("Find"),
+                               new BMessage(MSG_FIND)))
+               .TopView());
 
-       layoutView->SetViewColor(ui_color(B_PANEL_BACKGROUND_COLOR));
-
        fFindLabel->SetDivider(0.0);
 
        if (!findSelection) {

Modified: haiku/trunk/src/preferences/filetypes/ApplicationTypeWindow.cpp
===================================================================
--- haiku/trunk/src/preferences/filetypes/ApplicationTypeWindow.cpp     
2010-08-12 14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/preferences/filetypes/ApplicationTypeWindow.cpp     
2010-08-12 16:54:03 UTC (rev 38064)
@@ -23,6 +23,7 @@
 #include <GridLayoutBuilder.h>
 #include <GroupLayoutBuilder.h>
 #include <GroupView.h>
+#include <LayoutBuilder.h>
 #include <ListView.h>
 #include <Locale.h>
 #include <MenuBar.h>
@@ -308,7 +309,8 @@
                .Add(fSingleLaunchButton, 0, 0).Add(fArgsOnlyCheckBox, 1, 0)
                .Add(fMultipleLaunchButton, 0, 1).Add(fBackgroundAppCheckBox, 
1, 1)
                .Add(fExclusiveLaunchButton, 0, 2)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .View());
        flagsBox->SetLabel(fFlagsCheckBox);
 
        // "Icon" group
@@ -319,7 +321,8 @@
        fIconView->SetModificationMessage(new BMessage(kMsgIconChanged));
        iconBox->AddChild(BGroupLayoutBuilder(B_HORIZONTAL)
                .Add(fIconView)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .TopView());
 
        // "Supported Types" group
 
@@ -340,7 +343,8 @@
                new BMessage(kMsgRemoveType));
 
        fTypeIconView = new IconView("type icon");
-       BView* iconHolder = 
BGroupLayoutBuilder(B_HORIZONTAL).Add(fTypeIconView);
+       BGroupView* iconHolder = new BGroupView(B_HORIZONTAL);
+       iconHolder->AddChild(fTypeIconView);
        fTypeIconView->SetModificationMessage(new 
BMessage(kMsgTypeIconsChanged));
 
        typeBox->AddChild(BGridLayoutBuilder(padding, padding)
@@ -351,7 +355,8 @@
                .SetInsets(padding, padding, padding, padding)
                .SetColumnWeight(0, 3)
                .SetColumnWeight(1, 2)
-               .SetColumnWeight(2, 1));
+               .SetColumnWeight(2, 1)
+               .View());
        iconHolder->SetExplicitAlignment(
                BAlignment(B_ALIGN_CENTER, B_ALIGN_MIDDLE));
 
@@ -416,19 +421,22 @@
                .Add(longLabel, 0, 2)
                .Add(scrollView, 1, 2, 10, 3)
                .SetInsets(padding, padding, padding, padding)
-               .SetRowWeight(3, 3));
+               .SetRowWeight(3, 3)
+               .View());
 
        // put it all together
-       SetLayout(new BGroupLayout(B_VERTICAL));
-       AddChild(menuBar);
-       AddChild(BGroupLayoutBuilder(B_VERTICAL, padding)
-               .Add(fSignatureControl)
-               .Add(BGroupLayoutBuilder(B_HORIZONTAL, padding)
-                       .Add(flagsBox, 3)
-                       .Add(iconBox, 1))
-               .Add(typeBox)
-               .Add(versionBox)
-               .SetInsets(padding, padding, padding, padding));
+       BLayoutBuilder::Group<>(this, B_VERTICAL, 0)
+               .SetInsets(0, 0, 0, 0)
+               .Add(menuBar)
+               .AddGroup(B_VERTICAL, padding)
+                       .SetInsets(padding, padding, padding, padding)
+                       .Add(fSignatureControl)
+                       .AddGroup(B_HORIZONTAL, padding)
+                               .Add(flagsBox, 3)
+                               .Add(iconBox, 1)
+                               .End()
+                       .Add(typeBox)
+                       .Add(versionBox);
 
        SetKeyMenuBar(menuBar);
 

Modified: haiku/trunk/src/preferences/filetypes/ApplicationTypesWindow.cpp
===================================================================
--- haiku/trunk/src/preferences/filetypes/ApplicationTypesWindow.cpp    
2010-08-12 14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/preferences/filetypes/ApplicationTypesWindow.cpp    
2010-08-12 16:54:03 UTC (rev 38064)
@@ -117,12 +117,11 @@
        fAbortButton = new BButton("abort", B_TRANSLATE("Abort"),
                new BMessage(B_CANCEL));
 
-       SetLayout(new BGroupLayout(B_VERTICAL));
-       AddChild(BGroupLayoutBuilder(B_VERTICAL, 3.0f)
+       float padding = be_control_look->DefaultItemSpacing();
+       BLayoutBuilder::Group<>(this, B_VERTICAL, padding)
+               .SetInsets(padding, padding, padding, padding)
                .Add(fStatusBar)
-               .Add(fAbortButton)
-               .SetInsets(3.0f, 3.0f, 3.0f, 3.0f)
-       );
+               .Add(fAbortButton);
 
        // center on screen
        BScreen screen(this);
@@ -190,8 +189,6 @@
        BButton* button = new BButton("remove", B_TRANSLATE("Remove 
uninstalled"),
                new BMessage(kMsgRemoveUninstalled));
 
-       SetLayout(BGroupLayoutBuilder(B_HORIZONTAL));
-
        // "Information" group
 
        BBox* infoBox = new BBox((char*)NULL);
@@ -215,7 +212,8 @@
                .Add(fSignatureView->TextView(), 1, 1, 2)
                .Add(fPathView->LabelView(), 0, 2)
                .Add(fPathView->TextView(), 1, 2, 2)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .View());
 
        // "Version" group
 
@@ -238,7 +236,8 @@
                .Add(fVersionView->TextView(), 1, 0)
                .Add(fDescriptionLabel->LabelView(), 0, 1)
                .Add(fDescriptionView, 1, 1, 2, 2)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .View());
 
        // Launch and Tracker buttons
 
@@ -249,7 +248,8 @@
        fTrackerButton = new BButton(
                B_TRANSLATE("Show in Tracker" B_UTF8_ELLIPSIS));
 
-       AddChild(BLayoutBuilder::Group<>(B_HORIZONTAL, padding)
+
+       BLayoutBuilder::Group<>(this, B_HORIZONTAL, padding)
                .AddGroup(B_VERTICAL, padding, 3)
                        .Add(scrollView)
                        .AddGroup(B_HORIZONTAL)
@@ -257,19 +257,18 @@
                                .AddGlue()
                                .End()
                        .End()
-//                     .AddGlue())//, 3)
                .AddGroup(B_VERTICAL, padding)
                        .Add(infoBox)
                        .Add(versionBox)
                        .AddGroup(B_HORIZONTAL, padding)
-                               .AddGlue()
                                .Add(fEditButton)
                                .Add(fLaunchButton)
                                .Add(fTrackerButton)
+                               .AddGlue()
                                .End()
                        .AddGlue()
                        .End()
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding);
 
        BMimeType::StartWatching(this);
        _SetType(NULL);

Modified: haiku/trunk/src/preferences/filetypes/AttributeWindow.cpp
===================================================================
--- haiku/trunk/src/preferences/filetypes/AttributeWindow.cpp   2010-08-12 
14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/preferences/filetypes/AttributeWindow.cpp   2010-08-12 
16:54:03 UTC (rev 38064)
@@ -15,6 +15,7 @@
 #include <ControlLook.h>
 #include <GridLayoutBuilder.h>
 #include <GroupLayoutBuilder.h>
+#include <LayoutBuilder.h>
 #include <Locale.h>
 #include <MenuField.h>
 #include <MenuItem.h>
@@ -142,6 +143,10 @@
        BMenuField* typeMenuField = new BMenuField("types" , 
B_TRANSLATE("Type:"),
                fTypeMenu);
        typeMenuField->SetAlignment(B_ALIGN_RIGHT);
+       // we must set the color manually when adding a menuField directly
+       // into a window.
+       typeMenuField->SetViewColor(ui_color(B_PANEL_BACKGROUND_COLOR));
+       typeMenuField->SetLowColor(ui_color(B_PANEL_BACKGROUND_COLOR));
 
        fVisibleCheckBox = new BCheckBox("visible", B_TRANSLATE("Visible"),
                new BMessage(kMsgVisibilityChanged));
@@ -238,18 +243,18 @@
                new BMessage(B_QUIT_REQUESTED));
 
        BBox* visibleBox;
-       SetLayout(new BGroupLayout(B_VERTICAL));
-       AddChild(BGroupLayoutBuilder(B_VERTICAL, padding)
+       BLayoutBuilder::Group<>(this, B_VERTICAL, padding)
                .SetInsets(padding, padding, padding, padding)
-               .Add(BGridLayoutBuilder(padding, padding / 2)
+               .AddGrid(padding, padding / 2)
                        .Add(fPublicNameControl->CreateLabelLayoutItem(), 0, 0)
                        .Add(fPublicNameControl->CreateTextViewLayoutItem(), 1, 
0)
                        .Add(fAttributeControl->CreateLabelLayoutItem(), 0, 1)
                        .Add(fAttributeControl->CreateTextViewLayoutItem(), 1, 
1)
                        .Add(typeMenuField->CreateLabelLayoutItem(), 0, 2)
-                       .Add(typeMenuField->CreateMenuBarLayoutItem(), 1, 2))
+                       .Add(typeMenuField->CreateMenuBarLayoutItem(), 1, 2)
+                       .End()
                .Add(visibleBox = new BBox(B_FANCY_BORDER,
-                       BGridLayoutBuilder(padding, padding / 2)
+                       BLayoutBuilder::Grid<>(padding, padding / 2)
                                
.Add(fDisplayAsMenuField->CreateLabelLayoutItem(), 0, 0)
                                
.Add(fDisplayAsMenuField->CreateMenuBarLayoutItem(), 1, 0)
                                .Add(fEditableCheckBox, 3, 0)
@@ -259,12 +264,15 @@
                                .Add(fWidthControl->CreateTextViewLayoutItem(), 
1, 2, 3)
                                
.Add(fAlignmentMenuField->CreateLabelLayoutItem(), 0, 3)
                                
.Add(fAlignmentMenuField->CreateMenuBarLayoutItem(), 1, 3, 3)
-                               .SetInsets(padding, padding, padding, padding)))
-               .Add(BGroupLayoutBuilder(B_HORIZONTAL, padding)
+                               .SetInsets(padding, padding, padding, padding)
+                               .View())
+                       )
+               .AddGroup(B_HORIZONTAL, padding)
                        .Add(BSpaceLayoutItem::CreateGlue())
                        .Add(BSpaceLayoutItem::CreateGlue())
                        .Add(cancelButton)
-                       .Add(fAcceptButton)));
+                       .Add(fAcceptButton);
+
        visibleBox->SetLabel(fVisibleCheckBox);
 
        fAcceptButton->MakeDefault(true);

Modified: haiku/trunk/src/preferences/filetypes/ExtensionWindow.cpp
===================================================================
--- haiku/trunk/src/preferences/filetypes/ExtensionWindow.cpp   2010-08-12 
14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/preferences/filetypes/ExtensionWindow.cpp   2010-08-12 
16:54:03 UTC (rev 38064)
@@ -11,8 +11,7 @@
 #include <Button.h>
 #include <Catalog.h>
 #include <ControlLook.h>
-#include <GridLayoutBuilder.h>
-#include <GroupLayout.h>
+#include <LayoutBuilder.h>
 #include <Locale.h>
 #include <MenuField.h>
 #include <MenuItem.h>
@@ -120,10 +119,6 @@
        fMimeType(type.Type()),
        fExtension(extension)
 {
-       SetLayout(new BGroupLayout(B_VERTICAL));
-
-       float padding = be_control_look->DefaultItemSpacing();
-
        fExtensionControl = new BTextControl(B_TRANSLATE("Extension:"),
                extension, NULL);
        fExtensionControl->SetModificationMessage(
@@ -145,13 +140,12 @@
        BButton* button = new BButton(B_TRANSLATE("Cancel"),
                new BMessage(B_QUIT_REQUESTED));
 
-       AddChild(BGridLayoutBuilder(padding, padding)
-               .Add(fExtensionControl->CreateLabelLayoutItem(), 0, 0)
-               .Add(fExtensionControl->CreateTextViewLayoutItem(), 1, 0)
-               .Add(fAcceptButton, 0, 1)
-               .Add(button, 1, 1)
+       float padding = be_control_look->DefaultItemSpacing();
+       BLayoutBuilder::Grid<>(this, padding, padding)
                .SetInsets(padding, padding, padding, padding)
-       );
+               .AddTextControl(fExtensionControl, 0, 0)
+               .Add(fAcceptButton, 0, 1)
+               .Add(button, 1, 1);
 
        // omit the leading dot
        if (fExtension.ByteAt(0) == '.')

Modified: haiku/trunk/src/preferences/filetypes/FileTypeWindow.cpp
===================================================================
--- haiku/trunk/src/preferences/filetypes/FileTypeWindow.cpp    2010-08-12 
14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/preferences/filetypes/FileTypeWindow.cpp    2010-08-12 
16:54:03 UTC (rev 38064)
@@ -82,7 +82,8 @@
                .Add(fTypeControl, 0, 0, 2, 1)
                .Add(fSelectTypeButton, 0, 1)
                .Add(fSameTypeAsButton, 1, 1)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .View());
 
        // "Icon" group
 
@@ -91,7 +92,8 @@
        fIconView = new IconView("icon");
        iconBox->AddChild(BGroupLayoutBuilder(B_HORIZONTAL)
                .Add(fIconView)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .TopView());
 
        // "Preferred Application" group
 
@@ -118,14 +120,15 @@
                .Add(fPreferredField, 0, 0, 2, 1)
                .Add(fSelectAppButton, 0, 1)
                .Add(fSameAppAsButton, 1, 1)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .View());
 
        BLayoutBuilder::Group<>(this, B_HORIZONTAL, padding)
+               .SetInsets(padding, padding, padding, padding)
                .AddGroup(B_VERTICAL, padding)
                        .Add(fileTypeBox)
                        .Add(preferredBox)
                        .End()
-               .SetInsets(padding, padding, padding, padding)
                .Add(iconBox);
 
        fTypeControl->MakeFocus(true);

Modified: haiku/trunk/src/preferences/filetypes/FileTypesWindow.cpp
===================================================================
--- haiku/trunk/src/preferences/filetypes/FileTypesWindow.cpp   2010-08-12 
14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/preferences/filetypes/FileTypesWindow.cpp   2010-08-12 
16:54:03 UTC (rev 38064)
@@ -26,6 +26,7 @@
 #include <ControlLook.h>
 #include <GridLayoutBuilder.h>
 #include <GroupLayoutBuilder.h>
+#include <LayoutBuilder.h>
 #include <ListView.h>
 #include <Locale.h>
 #include <MenuBar.h>
@@ -302,8 +303,6 @@
        if (settings.FindBool("show_rule", &showRule) != B_OK)
                showRule = false;
 
-       SetLayout(new BGroupLayout(B_VERTICAL));
-
        float padding = be_control_look->DefaultItemSpacing();
        BAlignment labelAlignment = be_control_look->DefaultLabelAlignment();
        BAlignment fullAlignment(B_ALIGN_USE_FULL_WIDTH, 
B_ALIGN_USE_FULL_HEIGHT);
@@ -352,8 +351,6 @@
        item->SetTarget(this);
        menu->AddItem(item);
        menuBar->AddItem(menu);
-
-       AddChild(menuBar);
        menuBar->SetExplicitAlignment(BAlignment(B_ALIGN_LEFT, B_ALIGN_TOP));
 
        // MIME Types list
@@ -379,7 +376,8 @@
                .Add(BSpaceLayoutItem::CreateGlue(), 1)
                .Add(fIconView, 3)
                .Add(BSpaceLayoutItem::CreateGlue(), 1)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .TopView());
 
        // "File Recognition" group
 
@@ -413,7 +411,8 @@
        fRuleControl->SetAlignment(B_ALIGN_RIGHT, B_ALIGN_LEFT);
        fRuleControl->Hide();
 
-       BView* recognitionBoxGrid = BGridLayoutBuilder(padding, padding / 2)
+       BGridView* recognitionBoxGrid = new BGridView(padding, padding / 2);
+       BGridLayoutBuilder(recognitionBoxGrid)
                .Add(fExtensionLabel->LabelView(), 0, 0)
                .Add(scrollView, 0, 1, 2, 3)
                .Add(fAddExtensionButton, 2, 1)
@@ -444,7 +443,8 @@
                .Add(fTypeNameControl->CreateTextViewLayoutItem(), 1, 1, 2)
                .Add(fDescriptionControl->CreateLabelLayoutItem(), 0, 2)
                .Add(fDescriptionControl->CreateTextViewLayoutItem(), 1, 2, 2)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .View());
 
        // "Preferred Application" group
 
@@ -469,7 +469,8 @@
                .Add(fPreferredField)
                .Add(fSelectButton)
                .Add(fSameAsButton)
-               .SetInsets(padding, padding, padding, padding));
+               .SetInsets(padding, padding, padding, padding)
+               .TopView());
 
        // "Extra Attributes" group
 
@@ -497,27 +498,31 @@
                .Add(attributesScroller, 0, 0, 2, 3)
                .Add(fAddAttributeButton, 2, 0)
                .Add(fRemoveAttributeButton, 2, 1)
-               .SetInsets(padding, padding, padding, padding));
-
-       BView* topView = BGroupLayoutBuilder(B_HORIZONTAL, padding)
                .SetInsets(padding, padding, padding, padding)
-               .Add(BGroupLayoutBuilder(B_VERTICAL, padding)
-                       .Add(typeListScrollView)
-                       .Add(BGroupLayoutBuilder(B_HORIZONTAL, padding)
-                               .Add(addTypeButton)
-                               .Add(fRemoveTypeButton)
-                               .AddGlue()))
-               // Right side
-               .Add(BGroupLayoutBuilder(B_VERTICAL, padding)
-                       .Add(BGroupLayoutBuilder(B_HORIZONTAL, padding)
-                               .Add(fIconBox, 1).Add(fRecognitionBox, 3))
-                       .Add(fDescriptionBox)
-                       .Add(fPreferredBox)
-                       .Add(fAttributeBox, 5));
+               .View());
 
-       //topView->SetExplicitAlignment(fullAlignment);
-       //topView->SetViewColor(ui_color(B_PANEL_BACKGROUND_COLOR));
-       AddChild(topView);
+       BLayoutBuilder::Group<>(this, B_VERTICAL, 0)
+               .SetInsets(0, 0, 0, 0)
+               .Add(menuBar)
+               .AddGroup(B_HORIZONTAL, padding)
+                       .SetInsets(padding, padding, padding, padding)
+                       .AddGroup(B_VERTICAL, padding)
+                               .Add(typeListScrollView)
+                               .AddGroup(B_HORIZONTAL, padding)
+                                       .Add(addTypeButton)
+                                       .Add(fRemoveTypeButton)
+                                       .AddGlue()
+                                       .End()
+                               .End()
+                       // Right side
+                       .AddGroup(B_VERTICAL, padding)
+                               .AddGroup(B_HORIZONTAL, padding)
+                                       .Add(fIconBox, 1)
+                                       .Add(fRecognitionBox, 3)
+                                       .End()
+                               .Add(fDescriptionBox)
+                               .Add(fPreferredBox)
+                               .Add(fAttributeBox, 5);
 
        _SetType(NULL);
        _ShowSnifferRule(showRule);

Modified: haiku/trunk/src/preferences/fonts/FontSelectionView.cpp
===================================================================
--- haiku/trunk/src/preferences/fonts/FontSelectionView.cpp     2010-08-12 
14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/preferences/fonts/FontSelectionView.cpp     2010-08-12 
16:54:03 UTC (rev 38064)
@@ -121,6 +121,7 @@
        fPreviewBox->AddChild(BGroupLayoutBuilder(B_HORIZONTAL)
                .Add(fPreviewText)
                .SetInsets(5, 5, 5, 5)
+               .TopView()
        );
 }
 

Modified: haiku/trunk/src/preferences/fonts/MainWindow.cpp
===================================================================
--- haiku/trunk/src/preferences/fonts/MainWindow.cpp    2010-08-12 14:50:57 UTC 
(rev 38063)
+++ haiku/trunk/src/preferences/fonts/MainWindow.cpp    2010-08-12 16:54:03 UTC 
(rev 38064)
@@ -19,12 +19,11 @@
 #include <Button.h>
 #include <Box.h>
 #include <Catalog.h>
-#include <GridLayoutBuilder.h>
-#include <GroupLayoutBuilder.h>
+#include <ControlLook.h>
+#include <LayoutBuilder.h>
 #include <Locale.h>
 #include <MessageRunner.h>
 #include <Screen.h>
-#include <SpaceLayoutItem.h>
 #include <TabView.h>
 #include <TextView.h>
 
@@ -62,22 +61,17 @@
 
        fFontsView->UpdateFonts();
 
-       SetLayout(new BGroupLayout(B_VERTICAL));
-
-       const float kInset = 10;
-
-       AddChild(BGroupLayoutBuilder(B_VERTICAL)
+       const float kInset = be_control_look->DefaultItemSpacing();
+       BLayoutBuilder::Group<>(this, B_VERTICAL)
+               .SetInsets(kInset, kInset, kInset, kInset)
                .Add(box)
 //             .Add(tabView)
-               .Add(BSpaceLayoutItem::CreateVerticalStrut(kInset))
-               .Add(BGroupLayoutBuilder(B_HORIZONTAL)
+               .AddStrut(kInset)
+               .AddGroup(B_HORIZONTAL)
                        .Add(fDefaultsButton)
-                       .Add(BSpaceLayoutItem::CreateHorizontalStrut(kInset))
+                       .AddStrut(kInset)
                        .Add(fRevertButton)
-                       .Add(BSpaceLayoutItem::CreateGlue())
-               )
-               .SetInsets(kInset, kInset, kInset, kInset)
-       );
+                       .AddGlue();
 
        if (fSettings.WindowCorner() == BPoint(-1, -1)) {
                // center window on screen

Modified: haiku/trunk/src/preferences/locale/FormatSettingsView.cpp
===================================================================
--- haiku/trunk/src/preferences/locale/FormatSettingsView.cpp   2010-08-12 
14:50:57 UTC (rev 38063)
+++ haiku/trunk/src/preferences/locale/FormatSettingsView.cpp   2010-08-12 
16:54:03 UTC (rev 38064)
@@ -153,8 +153,6 @@
        BView("WindowsSettingsView", B_FRAME_EVENTS),
        fLocale(locale)
 {
-       SetLayout(new BGroupLayout(B_HORIZONTAL));
-
        fLongDateExampleView = new BStringView("", "");
 
        for (int i = 0; i < 4; i++) {
@@ -248,7 +246,8 @@
        formatBox->AddChild(BGroupLayoutBuilder(B_VERTICAL)
                .Add(beforeRadioButton)
                .Add(afterRadioButton)
-               .SetInsets(spacing, 0, spacing, 0));
+               .SetInsets(spacing, 0, spacing, 0)
+               .TopView());
 
        _UpdateExamples();
        _ParseDateFormat();
@@ -263,62 +262,60 @@
        fNumbersBox->SetLabel(B_TRANSLATE("Numbers"));
        fCurrencyBox->SetLabel(B_TRANSLATE("Currency"));
 
-       fDateBox->AddChild(BLayoutBuilder::Group<>(B_HORIZONTAL)
-               .AddGroup(B_VERTICAL, spacing / 2)
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(new BStringView("", B_TRANSLATE("Long 
format:")))
-                               .Add(fLongDateExampleView)
-                               .AddGlue()
-                               .End()
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(fLongDateMenu[0])
-                               .Add(fLongDateSeparator[0])
-                               .End()
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(fLongDateMenu[1])
-                               .Add(fLongDateSeparator[1])
-                               .End()
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(fLongDateMenu[2])
-                               .Add(fLongDateSeparator[2])
-                               .End()
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(fLongDateMenu[3])
-                               .Add(fLongDateSeparator[3])
-                               .End()
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(new BStringView("", B_TRANSLATE("Short 
format:")))
-                               .Add(fShortDateExampleView)
-                               .AddGlue()
-                               .End()
-                       .Add(fDateMenu[0])
-                       .Add(fDateMenu[1])
-                       .Add(fDateMenu[2])
+       fDateBox->AddChild(BLayoutBuilder::Group<>(B_VERTICAL, spacing / 2)
+               .SetInsets(spacing, spacing, spacing, spacing)
+               .AddGroup(B_HORIZONTAL, spacing)
+                       .Add(new BStringView("", B_TRANSLATE("Long format:")))
+                       .Add(fLongDateExampleView)
+                       .AddGlue()
                        .End()
-               .SetInsets(spacing, spacing, spacing, spacing));
+               .AddGroup(B_HORIZONTAL, spacing)
+                       .Add(fLongDateMenu[0])
+                       .Add(fLongDateSeparator[0])
+                       .End()
+               .AddGroup(B_HORIZONTAL, spacing)
+                       .Add(fLongDateMenu[1])
+                       .Add(fLongDateSeparator[1])
+                       .End()
+               .AddGroup(B_HORIZONTAL, spacing)
+                       .Add(fLongDateMenu[2])
+                       .Add(fLongDateSeparator[2])
+                       .End()
+               .AddGroup(B_HORIZONTAL, spacing)
+                       .Add(fLongDateMenu[3])
+                       .Add(fLongDateSeparator[3])
+                       .End()
+               .AddGroup(B_HORIZONTAL, spacing)
+                       .Add(new BStringView("", B_TRANSLATE("Short format:")))
+                       .Add(fShortDateExampleView)
+                       .AddGlue()
+                       .End()
+               .Add(fDateMenu[0])
+               .Add(fDateMenu[1])
+               .Add(fDateMenu[2])
+               .View());
 
-       fTimeBox->AddChild(BLayoutBuilder::Group<>(B_HORIZONTAL, spacing)
-               .AddGroup(B_VERTICAL, spacing / 2)
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(new BStringView("", B_TRANSLATE("Long 
format:")))
-                               .Add(fLongTimeExampleView)
-                               .AddGlue()
-                               .End()
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(new BStringView("", B_TRANSLATE("Short 
format:")))
-                               .Add(fShortTimeExampleView)
-                               .AddGlue()
-                               .End()
-                       .AddGroup(B_HORIZONTAL, spacing)
-                               .Add(f24HrRadioButton)
-                               .Add(f12HrRadioButton)
-                               .AddGlue()
-                               .End()
+       fTimeBox->AddChild(BLayoutBuilder::Group<>(B_VERTICAL, spacing / 2)
+               .SetInsets(spacing, spacing, spacing, spacing)
+               .AddGroup(B_HORIZONTAL, spacing)
+                       .Add(new BStringView("", B_TRANSLATE("Long format:")))
+                       .Add(fLongTimeExampleView)
+                       .AddGlue()
                        .End()
-               .SetInsets(spacing, spacing, spacing, spacing));
+               .AddGroup(B_HORIZONTAL, spacing)
+                       .Add(new BStringView("", B_TRANSLATE("Short format:")))
+                       .Add(fShortTimeExampleView)
+                       .AddGlue()
+                       .End()
+               .AddGroup(B_HORIZONTAL, spacing)

[... truncated: 182 lines follow ...]

Other related posts: