Browse Source

LibGUI+Applications: Use the new layout system in the settings screens

FrHun 3 years ago
parent
commit
a6ec2b9d0a

+ 4 - 8
Userland/Applications/DisplaySettings/BackgroundSettings.gml

@@ -28,22 +28,20 @@
 
         @GUI::IconView {
             name: "wallpaper_view"
+            preferred_width: "opportunistic_grow"
         }
 
         @GUI::Widget {
-            shrink_to_fit: true
+            preferred_width: "fit"
             layout: @GUI::VerticalBoxLayout {}
 
             @GUI::Button {
                 name: "wallpaper_open_button"
                 tooltip: "Select wallpaper from file system"
                 text: "Browse..."
-                shrink_to_fit: true
             }
 
-            @GUI::Widget {
-                fixed_height: 12
-            }
+            @GUI::Layout::Spacer {}
 
             @GUI::Label {
                 text: "Mode:"
@@ -55,9 +53,7 @@
                 name: "mode_combo"
             }
 
-            @GUI::Widget {
-                fixed_height: 12
-            }
+            @GUI::Layout::Spacer {}
 
             @GUI::Label {
                 text: "Color:"

+ 13 - 15
Userland/Applications/DisplaySettings/FontSettings.gml

@@ -6,7 +6,7 @@
     }
 
     @GUI::Widget {
-        shrink_to_fit: true
+        preferred_height: "fit"
         layout: @GUI::HorizontalBoxLayout {
             spacing: 6
         }
@@ -17,14 +17,13 @@
             text_alignment: "CenterLeft"
         }
 
-        @GUI::Frame {
+        @GUI::Label {
             background_role: "Base"
+            shadow: "Sunken"
+            shape: "Container"
+            thickness: 2
             fill_with_background_color: true
-            layout: @GUI::VerticalBoxLayout {}
-
-            @GUI::Label {
-                name: "default_font_label"
-            }
+            name: "default_font_label"
         }
 
         @GUI::Button {
@@ -35,7 +34,7 @@
     }
 
     @GUI::Widget {
-        shrink_to_fit: true
+        preferred_height: "fit"
         layout: @GUI::HorizontalBoxLayout {
             spacing: 6
         }
@@ -46,14 +45,13 @@
             text_alignment: "CenterLeft"
         }
 
-        @GUI::Frame {
+        @GUI::Label {
             background_role: "Base"
+            shadow: "Sunken"
+            shape: "Container"
+            thickness: 2
             fill_with_background_color: true
-            layout: @GUI::VerticalBoxLayout {}
-
-            @GUI::Label {
-                name: "fixed_width_font_label"
-            }
+            name: "fixed_width_font_label"
         }
 
         @GUI::Button {
@@ -63,5 +61,5 @@
         }
     }
 
-    @GUI::Widget {}
+    @GUI::Layout::Spacer {}
 }

+ 5 - 3
Userland/Applications/DisplaySettings/MonitorSettings.gml

@@ -15,7 +15,7 @@
     }
 
     @GUI::Widget {
-        shrink_to_fit: true
+        preferred_height: "fit"
         layout: @GUI::HorizontalBoxLayout {
             margins: [8, 8, 6, 16]
         }
@@ -38,7 +38,7 @@
         title: "Screen settings"
 
         @GUI::Widget {
-            shrink_to_fit: true
+            preferred_height: "fit"
             layout: @GUI::HorizontalBoxLayout {}
 
             @GUI::Label {
@@ -64,7 +64,7 @@
         }
 
         @GUI::Widget {
-            shrink_to_fit: true
+            preferred_height: "fit"
             layout: @GUI::HorizontalBoxLayout {}
 
             @GUI::Label {
@@ -84,6 +84,8 @@
                 text: "2x"
                 fixed_width: 50
             }
+
+            @GUI::Layout::Spacer {}
         }
     }
 }

+ 2 - 2
Userland/Applications/DisplaySettings/ThemesSettings.gml

@@ -23,13 +23,13 @@
         }
 
         @GUI::Widget {
-            shrink_to_fit: true
+            preferred_height: "fit"
             layout: @GUI::HorizontalBoxLayout {}
 
             @GUI::Label {
                 text: "Theme:"
                 text_alignment: "CenterLeft"
-                fixed_width: 95
+                preferred_width: 95
             }
 
             @GUI::ComboBox {

+ 1 - 1
Userland/Applications/MouseSettings/Theme.gml

@@ -12,7 +12,7 @@
         }
 
         @GUI::Widget {
-            shrink_to_fit: true
+            preferred_height: "fit"
             layout: @GUI::HorizontalBoxLayout {
                 spacing: 8
             }

+ 4 - 4
Userland/Applications/TerminalSettings/TerminalSettingsMain.gml

@@ -7,7 +7,7 @@
 
     @GUI::GroupBox {
         title: "Bell Mode"
-        shrink_to_fit: false
+        preferred_height: "fit"
         fixed_height: 160
         layout: @GUI::VerticalBoxLayout {
             margins: [16, 8, 8]
@@ -20,7 +20,7 @@
         }
 
         @GUI::Widget {
-            shrink_to_fit: true
+            preferred_height: "fit"
             layout: @GUI::VerticalBoxLayout {
                 spacing: 4
             }
@@ -44,7 +44,7 @@
 
     @GUI::GroupBox {
         title: "Scrollback Size (Lines)"
-        shrink_to_fit: true
+        preferred_height: "fit"
         layout: @GUI::VerticalBoxLayout {
             margins: [16, 8, 8]
         }
@@ -64,7 +64,7 @@
 
     @GUI::GroupBox {
         title: "Exit Behaviour"
-        shrink_to_fit: true
+        preferred_height: "fit"
         layout: @GUI::VerticalBoxLayout {
             margins: [16, 8, 8]
         }

+ 5 - 4
Userland/Applications/TerminalSettings/TerminalSettingsView.gml

@@ -7,7 +7,7 @@
 
     @GUI::GroupBox {
         title: "Background Opacity"
-        fixed_height: 70
+        preferred_height: "fit"
         layout: @GUI::VerticalBoxLayout {
             margins: [16, 8, 8]
             spacing: 16
@@ -23,7 +23,7 @@
 
     @GUI::GroupBox {
         title: "Terminal Font"
-        fixed_height: 100
+        preferred_height: "fit"
         layout: @GUI::VerticalBoxLayout {
             margins: [16, 8, 8]
             spacing: 16
@@ -35,13 +35,14 @@
         }
 
         @GUI::Widget {
-            shrink_to_fit: true
+            preferred_height: "fit"
             name: "terminal_font_selection"
             layout: @GUI::HorizontalBoxLayout {
                 spacing: 6
             }
 
             @GUI::Frame {
+                preferred_height: "fit"
                 background_role: "Base"
                 fill_with_background_color: true
                 layout: @GUI::VerticalBoxLayout {}
@@ -89,7 +90,7 @@
 
     @GUI::GroupBox {
         title: "Color Scheme"
-        fixed_height: 70
+        preferred_height: "fit"
         layout: @GUI::VerticalBoxLayout {
             margins: [16, 8, 8]
             spacing: 16

+ 1 - 1
Userland/Libraries/LibGUI/SettingsWindow.cpp

@@ -41,7 +41,7 @@ ErrorOr<NonnullRefPtr<SettingsWindow>> SettingsWindow::create(String title, Show
     window->m_tab_widget = TRY(main_widget->try_add<GUI::TabWidget>());
 
     auto button_container = TRY(main_widget->try_add<GUI::Widget>());
-    button_container->set_shrink_to_fit(true);
+    button_container->set_preferred_size({ SpecialDimension::Grow, SpecialDimension::Fit });
     (void)TRY(button_container->try_set_layout<GUI::HorizontalBoxLayout>());
     button_container->layout()->set_spacing(6);