From e11d17761846b04c8cd3d0fe1254f0a19bd3ae0e Mon Sep 17 00:00:00 2001 From: sin-ack Date: Tue, 17 Aug 2021 00:11:38 +0000 Subject: Userland+LibGUI: Add shorthand versions of the Margins constructor This allows for typing [8] instead of [8, 8, 8, 8] to specify the same margin on all edges, for example. The constructors follow CSS' style of specifying margins. The added constructors are: - Margins(int all): Sets the same margin on all edges. - Margins(int vertical, int horizontal): Sets the first argument to top and bottom margins, and the second argument to left and right margins. - Margins(int top, int vertical, int bottom): Sets the first argument to the top margin, the second argument to the left and right margins, and the third argument to the bottom margin. --- .../devel/templates/serenity-application/main.cpp | 2 +- Userland/Applets/Audio/main.cpp | 2 +- Userland/Applications/Assistant/main.cpp | 6 +-- Userland/Applications/Browser/BrowserWindow.gml | 2 +- Userland/Applications/Browser/DownloadWidget.cpp | 2 +- Userland/Applications/Browser/EditBookmark.gml | 2 +- .../Applications/Calculator/CalculatorWindow.gml | 2 +- Userland/Applications/Calendar/AddEventDialog.cpp | 4 +- .../CrashReporter/CrashReporterWindow.gml | 2 +- Userland/Applications/CrashReporter/main.cpp | 10 ++-- .../DisplaySettings/BackgroundSettings.gml | 2 +- .../DisplaySettings/DesktopSettings.gml | 8 ++-- .../Applications/DisplaySettings/FontSettings.gml | 2 +- .../DisplaySettings/MonitorSettings.gml | 4 +- Userland/Applications/DisplaySettings/main.cpp | 2 +- .../Applications/FileManager/DirectoryView.cpp | 2 +- .../FileManager/FileOperationProgress.gml | 2 +- .../Applications/FileManager/PropertiesWindow.cpp | 2 +- .../FileManager/PropertiesWindowGeneralTab.gml | 2 +- Userland/Applications/FileManager/main.cpp | 4 +- Userland/Applications/FontEditor/FontEditor.cpp | 4 +- .../Applications/FontEditor/NewFontDialogPage1.gml | 2 +- .../Applications/FontEditor/NewFontDialogPage2.gml | 6 +-- Userland/Applications/Help/main.cpp | 4 +- Userland/Applications/HexEditor/FindDialog.gml | 2 +- .../Applications/HexEditor/GoToOffsetDialog.gml | 6 +-- Userland/Applications/IRCClient/IRCAppWindow.cpp | 2 +- .../KeyboardMapper/KeyboardMapperWidget.cpp | 2 +- Userland/Applications/KeyboardSettings/main.cpp | 4 +- .../MailSettings/MailSettingsWindow.cpp | 2 +- .../MailSettings/MailSettingsWindow.gml | 6 +-- .../MouseSettings/MouseSettingsWindow.cpp | 2 +- .../MouseSettings/MouseSettingsWindow.gml | 12 ++--- Userland/Applications/PDFViewer/SidebarWidget.cpp | 4 +- Userland/Applications/Piano/MainWidget.cpp | 2 +- Userland/Applications/Piano/SamplerWidget.cpp | 2 +- .../PixelPaint/CreateNewGuideDialog.gml | 8 ++-- .../PixelPaint/CreateNewImageDialog.cpp | 2 +- .../PixelPaint/CreateNewLayerDialog.cpp | 2 +- Userland/Applications/PixelPaint/FilterParams.h | 2 +- .../PixelPaint/LayerPropertiesWidget.cpp | 2 +- .../Applications/PixelPaint/PixelPaintWindow.gml | 2 +- .../PixelPaint/ToolPropertiesWidget.cpp | 2 +- Userland/Applications/PixelPaint/ToolboxWidget.cpp | 2 +- Userland/Applications/Run/Run.gml | 4 +- .../Applications/Spreadsheet/CellTypeDialog.cpp | 16 +++---- .../Applications/Spreadsheet/CondFormatting.gml | 2 +- .../Applications/Spreadsheet/SpreadsheetView.cpp | 2 +- .../Applications/Spreadsheet/SpreadsheetWidget.cpp | 4 +- Userland/Applications/Spreadsheet/csv_export.gml | 8 ++-- Userland/Applications/Spreadsheet/csv_import.gml | 10 ++-- .../Spreadsheet/select_format_page.gml | 2 +- .../SystemMonitor/InterruptsWidget.cpp | 2 +- .../SystemMonitor/MemoryStatsWidget.cpp | 2 +- .../SystemMonitor/NetworkStatisticsWidget.cpp | 8 ++-- .../ProcessFileDescriptorMapWidget.cpp | 2 +- .../SystemMonitor/ProcessMemoryMapWidget.cpp | 2 +- .../SystemMonitor/ProcessStateWidget.cpp | 2 +- .../SystemMonitor/ProcessUnveiledPathsWidget.cpp | 2 +- .../SystemMonitor/ThreadStackWidget.cpp | 2 +- Userland/Applications/SystemMonitor/main.cpp | 20 ++++---- .../Terminal/TerminalSettingsWindow.gml | 10 ++-- Userland/Applications/Terminal/main.cpp | 4 +- .../Applications/TextEditor/TextEditorWindow.gml | 2 +- Userland/Demos/WidgetGallery/DemoWizardPage1.gml | 2 +- Userland/Demos/WidgetGallery/DemoWizardPage2.gml | 2 +- .../Demos/WidgetGallery/GalleryGML/BasicsTab.gml | 12 ++--- .../Demos/WidgetGallery/GalleryGML/CursorsTab.gml | 4 +- .../Demos/WidgetGallery/GalleryGML/IconsTab.gml | 4 +- .../Demos/WidgetGallery/GalleryGML/SlidersTab.gml | 12 ++--- Userland/Demos/WidgetGallery/GalleryGML/Window.gml | 2 +- .../Demos/WidgetGallery/GalleryGML/WizardsTab.gml | 4 +- .../Debugger/EvaluateExpressionDialog.cpp | 4 +- .../HackStudio/Dialogs/NewProjectDialog.gml | 2 +- Userland/DevTools/HackStudio/EditorWrapper.cpp | 2 +- Userland/DevTools/HackStudio/HackStudioWidget.cpp | 6 +-- Userland/DevTools/Profiler/main.cpp | 8 ++-- Userland/Games/2048/GameSizeDialog.cpp | 2 +- Userland/Games/Breakout/LevelSelectDialog.cpp | 2 +- Userland/Games/GameOfLife/main.cpp | 2 +- Userland/Games/Hearts/Game.cpp | 2 +- Userland/Games/Hearts/SettingsDialog.cpp | 2 +- Userland/Libraries/LibGUI/AboutDialog.cpp | 4 +- Userland/Libraries/LibGUI/ColorPicker.cpp | 8 ++-- Userland/Libraries/LibGUI/EmojiInputDialog.cpp | 2 +- Userland/Libraries/LibGUI/FilePickerDialog.gml | 6 +-- Userland/Libraries/LibGUI/Frame.cpp | 2 +- Userland/Libraries/LibGUI/InputBox.cpp | 2 +- Userland/Libraries/LibGUI/Margins.h | 53 +++++++++++++++++----- Userland/Libraries/LibGUI/MessageBox.cpp | 2 +- Userland/Libraries/LibGUI/MultiView.cpp | 2 +- Userland/Libraries/LibGUI/PasswordInputDialog.gml | 2 +- Userland/Libraries/LibGUI/ProcessChooser.cpp | 4 +- Userland/Libraries/LibGUI/Statusbar.cpp | 2 +- Userland/Libraries/LibGUI/ToolbarContainer.cpp | 2 +- .../Libraries/LibGUI/Wizards/CoverWizardPage.cpp | 2 +- Userland/Libraries/LibGUI/Wizards/WizardDialog.cpp | 2 +- Userland/Libraries/LibGUI/Wizards/WizardPage.cpp | 4 +- .../NotificationServer/NotificationWindow.cpp | 2 +- Userland/Services/Taskbar/ClockWidget.cpp | 2 +- Userland/Services/Taskbar/ShutdownDialog.cpp | 2 +- 101 files changed, 232 insertions(+), 201 deletions(-) diff --git a/Base/res/devel/templates/serenity-application/main.cpp b/Base/res/devel/templates/serenity-application/main.cpp index d3e728bc71..bcb7bb1669 100644 --- a/Base/res/devel/templates/serenity-application/main.cpp +++ b/Base/res/devel/templates/serenity-application/main.cpp @@ -28,7 +28,7 @@ int main(int argc, char** argv) main_widget.set_fill_with_background_color(true); auto& layout = main_widget.set_layout(); - layout.set_margins({ 16, 16, 16, 16 }); + layout.set_margins(16); auto& button = main_widget.add("Click me!"); button.on_click = [&](auto) { diff --git a/Userland/Applets/Audio/main.cpp b/Userland/Applets/Audio/main.cpp index b62dde9e91..a39611ea85 100644 --- a/Userland/Applets/Audio/main.cpp +++ b/Userland/Applets/Audio/main.cpp @@ -63,7 +63,7 @@ public: m_root_container = m_slider_window->set_main_widget(); m_root_container->set_fill_with_background_color(true); m_root_container->set_layout(); - m_root_container->layout()->set_margins({ 4, 0, 4, 0 }); + m_root_container->layout()->set_margins({ 4, 0 }); m_root_container->layout()->set_spacing(0); m_root_container->set_frame_thickness(2); m_root_container->set_frame_shape(Gfx::FrameShape::Container); diff --git a/Userland/Applications/Assistant/main.cpp b/Userland/Applications/Assistant/main.cpp index 1d1de4bb62..0f6a7c01fa 100644 --- a/Userland/Applications/Assistant/main.cpp +++ b/Userland/Applications/Assistant/main.cpp @@ -39,7 +39,7 @@ public: { auto& layout = set_layout(); layout.set_spacing(12); - layout.set_margins({ 4, 4, 4, 4 }); + layout.set_margins(4); m_image = add(); @@ -221,12 +221,12 @@ int main(int argc, char** argv) container.set_fill_with_background_color(true); container.set_frame_shadow(Gfx::FrameShadow::Raised); auto& layout = container.set_layout(); - layout.set_margins({ 8, 8, 0, 8 }); + layout.set_margins({ 8, 8, 0 }); auto& text_box = container.add(); auto& results_container = container.add(); auto& results_layout = results_container.set_layout(); - results_layout.set_margins({ 10, 0, 10, 0 }); + results_layout.set_margins({ 10, 0 }); auto mark_selected_item = [&]() { for (size_t i = 0; i < app_state.visible_result_count; ++i) { diff --git a/Userland/Applications/Browser/BrowserWindow.gml b/Userland/Applications/Browser/BrowserWindow.gml index 39a6b05f84..d8360a9583 100644 --- a/Userland/Applications/Browser/BrowserWindow.gml +++ b/Userland/Applications/Browser/BrowserWindow.gml @@ -12,7 +12,7 @@ @GUI::TabWidget { name: "tab_widget" - container_margins: [0, 0, 0, 0] + container_margins: [0] uniform_tabs: true text_alignment: "CenterLeft" } diff --git a/Userland/Applications/Browser/DownloadWidget.cpp b/Userland/Applications/Browser/DownloadWidget.cpp index 39b3835677..d0c0e92a96 100644 --- a/Userland/Applications/Browser/DownloadWidget.cpp +++ b/Userland/Applications/Browser/DownloadWidget.cpp @@ -61,7 +61,7 @@ DownloadWidget::DownloadWidget(const URL& url) set_fill_with_background_color(true); auto& layout = set_layout(); - layout.set_margins({ 4, 4, 4, 4 }); + layout.set_margins(4); auto& animation_container = add(); animation_container.set_fixed_height(32); diff --git a/Userland/Applications/Browser/EditBookmark.gml b/Userland/Applications/Browser/EditBookmark.gml index e7f7dd7169..86134dfe8e 100644 --- a/Userland/Applications/Browser/EditBookmark.gml +++ b/Userland/Applications/Browser/EditBookmark.gml @@ -4,7 +4,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } diff --git a/Userland/Applications/Calculator/CalculatorWindow.gml b/Userland/Applications/Calculator/CalculatorWindow.gml index 09274ab223..fccca448e5 100644 --- a/Userland/Applications/Calculator/CalculatorWindow.gml +++ b/Userland/Applications/Calculator/CalculatorWindow.gml @@ -13,7 +13,7 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::TextBox { diff --git a/Userland/Applications/Calendar/AddEventDialog.cpp b/Userland/Applications/Calendar/AddEventDialog.cpp index 547298730c..010a89b19a 100644 --- a/Userland/Applications/Calendar/AddEventDialog.cpp +++ b/Userland/Applications/Calendar/AddEventDialog.cpp @@ -39,7 +39,7 @@ AddEventDialog::AddEventDialog(Core::DateTime date_time, Window* parent_window) auto& top_container = widget.add(); top_container.set_layout(); top_container.set_fixed_height(45); - top_container.layout()->set_margins({ 4, 4, 4, 4 }); + top_container.layout()->set_margins(4); auto& add_label = top_container.add("Add title & date:"); add_label.set_text_alignment(Gfx::TextAlignment::CenterLeft); @@ -52,7 +52,7 @@ AddEventDialog::AddEventDialog(Core::DateTime date_time, Window* parent_window) auto& middle_container = widget.add(); middle_container.set_layout(); middle_container.set_fixed_height(25); - middle_container.layout()->set_margins({ 4, 4, 4, 4 }); + middle_container.layout()->set_margins(4); auto& starting_month_combo = middle_container.add(); starting_month_combo.set_only_allow_values_from_model(true); diff --git a/Userland/Applications/CrashReporter/CrashReporterWindow.gml b/Userland/Applications/CrashReporter/CrashReporterWindow.gml index 2e6a8abfa2..5bf6d547f6 100644 --- a/Userland/Applications/CrashReporter/CrashReporterWindow.gml +++ b/Userland/Applications/CrashReporter/CrashReporterWindow.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [5, 5, 5, 5] + margins: [5] } @GUI::Widget { diff --git a/Userland/Applications/CrashReporter/main.cpp b/Userland/Applications/CrashReporter/main.cpp index b8bb70df15..d14f37f0a1 100644 --- a/Userland/Applications/CrashReporter/main.cpp +++ b/Userland/Applications/CrashReporter/main.cpp @@ -225,7 +225,7 @@ int main(int argc, char** argv) auto& backtrace_tab = tab_widget.add_tab("Backtrace"); backtrace_tab.set_layout(); - backtrace_tab.layout()->set_margins({ 4, 4, 4, 4 }); + backtrace_tab.layout()->set_margins(4); auto& backtrace_label = backtrace_tab.add("A backtrace for each thread alive during the crash is listed below:"); backtrace_label.set_text_alignment(Gfx::TextAlignment::CenterLeft); @@ -236,7 +236,7 @@ int main(int argc, char** argv) for (auto& backtrace : thread_backtraces) { auto& backtrace_text_editor = backtrace_tab_widget.add_tab(backtrace.title); backtrace_text_editor.set_layout(); - backtrace_text_editor.layout()->set_margins({ 4, 4, 4, 4 }); + backtrace_text_editor.layout()->set_margins(4); backtrace_text_editor.set_text(backtrace.text); backtrace_text_editor.set_mode(GUI::TextEditor::Mode::ReadOnly); backtrace_text_editor.set_should_hide_unnecessary_scrollbars(true); @@ -244,7 +244,7 @@ int main(int argc, char** argv) auto& cpu_registers_tab = tab_widget.add_tab("CPU Registers"); cpu_registers_tab.set_layout(); - cpu_registers_tab.layout()->set_margins({ 4, 4, 4, 4 }); + cpu_registers_tab.layout()->set_margins(4); auto& cpu_registers_label = cpu_registers_tab.add("The CPU register state for each thread alive during the crash is listed below:"); cpu_registers_label.set_text_alignment(Gfx::TextAlignment::CenterLeft); @@ -255,7 +255,7 @@ int main(int argc, char** argv) for (auto& cpu_registers : thread_cpu_registers) { auto& cpu_registers_text_editor = cpu_registers_tab_widget.add_tab(cpu_registers.title); cpu_registers_text_editor.set_layout(); - cpu_registers_text_editor.layout()->set_margins({ 4, 4, 4, 4 }); + cpu_registers_text_editor.layout()->set_margins(4); cpu_registers_text_editor.set_text(cpu_registers.text); cpu_registers_text_editor.set_mode(GUI::TextEditor::Mode::ReadOnly); cpu_registers_text_editor.set_should_hide_unnecessary_scrollbars(true); @@ -263,7 +263,7 @@ int main(int argc, char** argv) auto& environment_tab = tab_widget.add_tab("Environment"); environment_tab.set_layout(); - environment_tab.layout()->set_margins({ 4, 4, 4, 4 }); + environment_tab.layout()->set_margins(4); auto& environment_text_editor = environment_tab.add(); environment_text_editor.set_text(String::join("\n", environment)); diff --git a/Userland/Applications/DisplaySettings/BackgroundSettings.gml b/Userland/Applications/DisplaySettings/BackgroundSettings.gml index 125a2923d1..8b06a18e3a 100644 --- a/Userland/Applications/DisplaySettings/BackgroundSettings.gml +++ b/Userland/Applications/DisplaySettings/BackgroundSettings.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @DisplaySettings::MonitorWidget { diff --git a/Userland/Applications/DisplaySettings/DesktopSettings.gml b/Userland/Applications/DisplaySettings/DesktopSettings.gml index 1d753cd6cb..4dc51279e4 100644 --- a/Userland/Applications/DisplaySettings/DesktopSettings.gml +++ b/Userland/Applications/DisplaySettings/DesktopSettings.gml @@ -2,12 +2,12 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::GroupBox { layout: @GUI::VerticalBoxLayout { - margins: [24, 16, 6, 16] + margins: [24, 16, 6] } title: "Workspaces" @@ -17,7 +17,7 @@ fixed_height: 32 layout: @GUI::HorizontalBoxLayout { - margins: [6, 6, 6, 6] + margins: [6] } @GUI::Label { @@ -57,7 +57,7 @@ } @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [6, 6, 6, 6] + margins: [6] } @GUI::Label { text: "Use the Ctrl+Alt+Arrow hotkeys to move between workspaces." diff --git a/Userland/Applications/DisplaySettings/FontSettings.gml b/Userland/Applications/DisplaySettings/FontSettings.gml index c4ebe09fb7..dadda2e083 100644 --- a/Userland/Applications/DisplaySettings/FontSettings.gml +++ b/Userland/Applications/DisplaySettings/FontSettings.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] spacing: 8 } diff --git a/Userland/Applications/DisplaySettings/MonitorSettings.gml b/Userland/Applications/DisplaySettings/MonitorSettings.gml index 7984e9ea91..464d2d27db 100644 --- a/Userland/Applications/DisplaySettings/MonitorSettings.gml +++ b/Userland/Applications/DisplaySettings/MonitorSettings.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @DisplaySettings::MonitorWidget { @@ -34,7 +34,7 @@ @GUI::GroupBox { layout: @GUI::VerticalBoxLayout { - margins: [24, 16, 6, 16] + margins: [24, 16, 6] } title: "Screen settings" diff --git a/Userland/Applications/DisplaySettings/main.cpp b/Userland/Applications/DisplaySettings/main.cpp index b19cfc79d6..6d154f423a 100644 --- a/Userland/Applications/DisplaySettings/main.cpp +++ b/Userland/Applications/DisplaySettings/main.cpp @@ -45,7 +45,7 @@ int main(int argc, char** argv) auto& main_widget = window->set_main_widget(); main_widget.set_fill_with_background_color(true); main_widget.set_layout(); - main_widget.layout()->set_margins({ 4, 4, 4, 4 }); + main_widget.layout()->set_margins(4); main_widget.layout()->set_spacing(6); auto& tab_widget = main_widget.add(); diff --git a/Userland/Applications/FileManager/DirectoryView.cpp b/Userland/Applications/FileManager/DirectoryView.cpp index b576ba49e0..72a740e5b4 100644 --- a/Userland/Applications/FileManager/DirectoryView.cpp +++ b/Userland/Applications/FileManager/DirectoryView.cpp @@ -134,7 +134,7 @@ DirectoryView::DirectoryView(Mode mode) , m_sorting_model(GUI::SortingProxyModel::create(m_model)) { set_active_widget(nullptr); - set_content_margins({ 2, 2, 2, 2 }); + set_content_margins(2); setup_actions(); diff --git a/Userland/Applications/FileManager/FileOperationProgress.gml b/Userland/Applications/FileManager/FileOperationProgress.gml index f90ca68e8a..77d0c3dc13 100644 --- a/Userland/Applications/FileManager/FileOperationProgress.gml +++ b/Userland/Applications/FileManager/FileOperationProgress.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::Widget { diff --git a/Userland/Applications/FileManager/PropertiesWindow.cpp b/Userland/Applications/FileManager/PropertiesWindow.cpp index 13a9ba6b2a..64d3bb413a 100644 --- a/Userland/Applications/FileManager/PropertiesWindow.cpp +++ b/Userland/Applications/FileManager/PropertiesWindow.cpp @@ -32,7 +32,7 @@ PropertiesWindow::PropertiesWindow(String const& path, bool disable_rename, Wind auto& main_widget = set_main_widget(); main_widget.set_layout(); - main_widget.layout()->set_margins({ 4, 4, 4, 4 }); + main_widget.layout()->set_margins(4); main_widget.set_fill_with_background_color(true); set_rect({ 0, 0, 360, 420 }); diff --git a/Userland/Applications/FileManager/PropertiesWindowGeneralTab.gml b/Userland/Applications/FileManager/PropertiesWindowGeneralTab.gml index 142082911c..cb03b22cf1 100644 --- a/Userland/Applications/FileManager/PropertiesWindowGeneralTab.gml +++ b/Userland/Applications/FileManager/PropertiesWindowGeneralTab.gml @@ -1,6 +1,6 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [8, 12, 8, 12] + margins: [8, 12] spacing: 10 } diff --git a/Userland/Applications/FileManager/main.cpp b/Userland/Applications/FileManager/main.cpp index 3c37b36cc9..7bc98ce3d4 100644 --- a/Userland/Applications/FileManager/main.cpp +++ b/Userland/Applications/FileManager/main.cpp @@ -443,12 +443,12 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio auto& toolbar_container = *widget.find_descendant_of_type_named("toolbar_container"); auto& main_toolbar = *widget.find_descendant_of_type_named("main_toolbar"); auto& location_toolbar = *widget.find_descendant_of_type_named("location_toolbar"); - location_toolbar.layout()->set_margins({ 3, 6, 3, 6 }); + location_toolbar.layout()->set_margins({ 3, 6 }); auto& location_textbox = *widget.find_descendant_of_type_named("location_textbox"); auto& breadcrumb_toolbar = *widget.find_descendant_of_type_named("breadcrumb_toolbar"); - breadcrumb_toolbar.layout()->set_margins({ 0, 6, 0, 6 }); + breadcrumb_toolbar.layout()->set_margins({ 0, 6 }); auto& breadcrumbbar = *widget.find_descendant_of_type_named("breadcrumbbar"); auto& splitter = *widget.find_descendant_of_type_named("splitter"); diff --git a/Userland/Applications/FontEditor/FontEditor.cpp b/Userland/Applications/FontEditor/FontEditor.cpp index 73acf36f61..8dea764b03 100644 --- a/Userland/Applications/FontEditor/FontEditor.cpp +++ b/Userland/Applications/FontEditor/FontEditor.cpp @@ -60,12 +60,12 @@ static RefPtr create_font_preview_window(FontEditorWidget& editor) auto& main_widget = window->set_main_widget(); main_widget.set_fill_with_background_color(true); main_widget.set_layout(); - main_widget.layout()->set_margins({ 2, 2, 2, 2 }); + main_widget.layout()->set_margins(2); main_widget.layout()->set_spacing(4); auto& preview_box = main_widget.add(); preview_box.set_layout(); - preview_box.layout()->set_margins({ 8, 8, 8, 8 }); + preview_box.layout()->set_margins(8); auto& preview_label = preview_box.add(); preview_label.set_font(editor.edited_font()); diff --git a/Userland/Applications/FontEditor/NewFontDialogPage1.gml b/Userland/Applications/FontEditor/NewFontDialogPage1.gml index 0f560d1c2c..2f372622dc 100644 --- a/Userland/Applications/FontEditor/NewFontDialogPage1.gml +++ b/Userland/Applications/FontEditor/NewFontDialogPage1.gml @@ -1,6 +1,6 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [20, 20, 20, 20] + margins: [20] } @GUI::Widget { diff --git a/Userland/Applications/FontEditor/NewFontDialogPage2.gml b/Userland/Applications/FontEditor/NewFontDialogPage2.gml index e9ebf28656..266741bb18 100644 --- a/Userland/Applications/FontEditor/NewFontDialogPage2.gml +++ b/Userland/Applications/FontEditor/NewFontDialogPage2.gml @@ -1,6 +1,6 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [20, 20, 20, 20] + margins: [20] } @GUI::Widget { @@ -11,7 +11,7 @@ title: "Metadata" fixed_width: 200 layout: @GUI::VerticalBoxLayout { - margins: [16, 8, 8, 8] + margins: [16, 8, 8] } @GUI::Widget { @@ -125,7 +125,7 @@ @GUI::Widget { name: "glyph_editor_container" layout: @GUI::VerticalBoxLayout { - margins: [5, 0, 0, 0] + margins: [5, 0, 0] } } diff --git a/Userland/Applications/Help/main.cpp b/Userland/Applications/Help/main.cpp index 3f88f099c2..243284c4b0 100644 --- a/Userland/Applications/Help/main.cpp +++ b/Userland/Applications/Help/main.cpp @@ -96,11 +96,11 @@ int main(int argc, char* argv[]) auto& left_tab_bar = splitter.add(); auto& tree_view_container = left_tab_bar.add_tab("Browse"); tree_view_container.set_layout(); - tree_view_container.layout()->set_margins({ 4, 4, 4, 4 }); + tree_view_container.layout()->set_margins(4); auto& tree_view = tree_view_container.add(); auto& search_view = left_tab_bar.add_tab("Search"); search_view.set_layout(); - search_view.layout()->set_margins({ 4, 4, 4, 4 }); + search_view.layout()->set_margins(4); auto& search_box = search_view.add(); auto& search_list_view = search_view.add(); search_box.set_fixed_height(20); diff --git a/Userland/Applications/HexEditor/FindDialog.gml b/Userland/Applications/HexEditor/FindDialog.gml index 57b1c9cfa9..f3b9aac85f 100644 --- a/Userland/Applications/HexEditor/FindDialog.gml +++ b/Userland/Applications/HexEditor/FindDialog.gml @@ -6,7 +6,7 @@ layout: @GUI::VerticalBoxLayout { spacing: 2 - margins: [4, 4, 4, 4] + margins: [4] } @GUI::Widget { diff --git a/Userland/Applications/HexEditor/GoToOffsetDialog.gml b/Userland/Applications/HexEditor/GoToOffsetDialog.gml index 3234f5dc20..1828e782e3 100644 --- a/Userland/Applications/HexEditor/GoToOffsetDialog.gml +++ b/Userland/Applications/HexEditor/GoToOffsetDialog.gml @@ -6,13 +6,13 @@ layout: @GUI::VerticalBoxLayout { spacing: 2 - margins: [0, 0, 0, 0] + margins: [0] } @GUI::Widget { layout: @GUI::HorizontalBoxLayout { spacing: 2 - margins: [2, 2, 2, 2] + margins: [2] } @GUI::Label { @@ -41,7 +41,7 @@ @GUI::Widget { layout: @GUI::HorizontalBoxLayout { spacing: 2 - margins: [2, 2, 2, 2] + margins: [2] } @GUI::Label { diff --git a/Userland/Applications/IRCClient/IRCAppWindow.cpp b/Userland/Applications/IRCClient/IRCAppWindow.cpp index e24dab9d30..ec64224a0d 100644 --- a/Userland/Applications/IRCClient/IRCAppWindow.cpp +++ b/Userland/Applications/IRCClient/IRCAppWindow.cpp @@ -298,7 +298,7 @@ void IRCAppWindow::setup_widgets() auto& outer_container = widget.add(); outer_container.set_layout(); - outer_container.layout()->set_margins({ 0, 2, 2, 2 }); + outer_container.layout()->set_margins({ 0, 2, 2 }); auto& horizontal_container = outer_container.add(); diff --git a/Userland/Applications/KeyboardMapper/KeyboardMapperWidget.cpp b/Userland/Applications/KeyboardMapper/KeyboardMapperWidget.cpp index b40c233b0d..71e6cfa053 100644 --- a/Userland/Applications/KeyboardMapper/KeyboardMapperWidget.cpp +++ b/Userland/Applications/KeyboardMapper/KeyboardMapperWidget.cpp @@ -28,7 +28,7 @@ void KeyboardMapperWidget::create_frame() { set_fill_with_background_color(true); set_layout(); - layout()->set_margins({ 4, 4, 4, 4 }); + layout()->set_margins(4); auto& main_widget = add(); main_widget.set_relative_rect(0, 0, 200, 200); diff --git a/Userland/Applications/KeyboardSettings/main.cpp b/Userland/Applications/KeyboardSettings/main.cpp index 411e6b9536..091621cfde 100644 --- a/Userland/Applications/KeyboardSettings/main.cpp +++ b/Userland/Applications/KeyboardSettings/main.cpp @@ -109,7 +109,7 @@ int main(int argc, char** argv) root_widget.set_layout(); root_widget.set_fill_with_background_color(true); root_widget.layout()->set_spacing(0); - root_widget.layout()->set_margins({ 4, 4, 4, 4 }); + root_widget.layout()->set_margins(4); auto& character_map_file_selection_container = root_widget.add(); character_map_file_selection_container.set_layout(); @@ -154,7 +154,7 @@ int main(int argc, char** argv) bottom_widget.set_layout(); bottom_widget.layout()->add_spacer(); bottom_widget.set_fixed_height(30); - bottom_widget.set_content_margins({ 4, 0, 4, 0 }); + bottom_widget.set_content_margins({ 4, 0 }); auto& ok_button = bottom_widget.add(); ok_button.set_text("OK"); diff --git a/Userland/Applications/MailSettings/MailSettingsWindow.cpp b/Userland/Applications/MailSettings/MailSettingsWindow.cpp index 45a4754ef7..b3c7aeb893 100644 --- a/Userland/Applications/MailSettings/MailSettingsWindow.cpp +++ b/Userland/Applications/MailSettings/MailSettingsWindow.cpp @@ -66,7 +66,7 @@ MailSettingsWindow::MailSettingsWindow() auto& main_widget = set_main_widget(); main_widget.set_fill_with_background_color(true); main_widget.set_layout(); - main_widget.layout()->set_margins({ 4, 4, 4, 4 }); + main_widget.layout()->set_margins(4); main_widget.layout()->set_spacing(6); auto& tab_widget = main_widget.add(); diff --git a/Userland/Applications/MailSettings/MailSettingsWindow.gml b/Userland/Applications/MailSettings/MailSettingsWindow.gml index 8e6a61e3bf..410445fe20 100644 --- a/Userland/Applications/MailSettings/MailSettingsWindow.gml +++ b/Userland/Applications/MailSettings/MailSettingsWindow.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [10, 10, 10, 10] + margins: [10] spacing: 5 } @@ -11,7 +11,7 @@ fixed_height: 170 layout: @GUI::VerticalBoxLayout { - margins: [16, 8, 8, 8] + margins: [16, 8, 8] spacing: 2 } @@ -101,7 +101,7 @@ fixed_height: 110 layout: @GUI::VerticalBoxLayout { - margins: [16, 8, 8, 8] + margins: [16, 8, 8] spacing: 2 } diff --git a/Userland/Applications/MouseSettings/MouseSettingsWindow.cpp b/Userland/Applications/MouseSettings/MouseSettingsWindow.cpp index 85b26c05e2..bf170a46d7 100644 --- a/Userland/Applications/MouseSettings/MouseSettingsWindow.cpp +++ b/Userland/Applications/MouseSettings/MouseSettingsWindow.cpp @@ -44,7 +44,7 @@ MouseSettingsWindow::MouseSettingsWindow() auto& main_widget = set_main_widget(); main_widget.set_fill_with_background_color(true); main_widget.set_layout(); - main_widget.layout()->set_margins({ 4, 4, 4, 4 }); + main_widget.layout()->set_margins(4); main_widget.layout()->set_spacing(6); auto& tab_widget = main_widget.add(); diff --git a/Userland/Applications/MouseSettings/MouseSettingsWindow.gml b/Userland/Applications/MouseSettings/MouseSettingsWindow.gml index 3731fecceb..1ee8e37b93 100644 --- a/Userland/Applications/MouseSettings/MouseSettingsWindow.gml +++ b/Userland/Applications/MouseSettings/MouseSettingsWindow.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [10, 10, 10, 10] + margins: [10] spacing: 5 } @@ -11,7 +11,7 @@ fixed_height: 110 layout: @GUI::VerticalBoxLayout { - margins: [16, 8, 8, 8] + margins: [16, 8, 8] spacing: 2 } @@ -60,7 +60,7 @@ fixed_height: 110 layout: @GUI::VerticalBoxLayout { - margins: [16, 8, 8, 8] + margins: [16, 8, 8] spacing: 2 } @@ -83,7 +83,7 @@ @GUI::Widget { layout: @GUI::HorizontalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] spacing: 8 } @@ -114,7 +114,7 @@ fixed_height: 110 layout: @GUI::VerticalBoxLayout { - margins: [16, 8, 8, 8] + margins: [16, 8, 8] spacing: 2 } @@ -137,7 +137,7 @@ @GUI::Widget { layout: @GUI::HorizontalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] spacing: 8 } diff --git a/Userland/Applications/PDFViewer/SidebarWidget.cpp b/Userland/Applications/PDFViewer/SidebarWidget.cpp index 93bc4f8c42..a2740f3036 100644 --- a/Userland/Applications/PDFViewer/SidebarWidget.cpp +++ b/Userland/Applications/PDFViewer/SidebarWidget.cpp @@ -18,14 +18,14 @@ SidebarWidget::SidebarWidget() auto& outline_container = tab_bar.add_tab("Outline"); outline_container.set_layout(); - outline_container.layout()->set_margins({ 4, 4, 4, 4 }); + outline_container.layout()->set_margins(4); m_outline_tree_view = outline_container.add(); m_outline_tree_view->set_activates_on_selection(true); auto& thumbnails_container = tab_bar.add_tab("Thumbnails"); thumbnails_container.set_layout(); - thumbnails_container.layout()->set_margins({ 4, 4, 4, 4 }); + thumbnails_container.layout()->set_margins(4); // FIXME: Add thumbnail previews } diff --git a/Userland/Applications/Piano/MainWidget.cpp b/Userland/Applications/Piano/MainWidget.cpp index 296c0180c0..d9f95956ff 100644 --- a/Userland/Applications/Piano/MainWidget.cpp +++ b/Userland/Applications/Piano/MainWidget.cpp @@ -24,7 +24,7 @@ MainWidget::MainWidget(TrackManager& track_manager, AudioPlayerLoop& loop) { set_layout(); layout()->set_spacing(2); - layout()->set_margins({ 2, 2, 2, 2 }); + layout()->set_margins(2); set_fill_with_background_color(true); m_wave_widget = add(track_manager); diff --git a/Userland/Applications/Piano/SamplerWidget.cpp b/Userland/Applications/Piano/SamplerWidget.cpp index c9215ec08a..87b8cdf327 100644 --- a/Userland/Applications/Piano/SamplerWidget.cpp +++ b/Userland/Applications/Piano/SamplerWidget.cpp @@ -72,7 +72,7 @@ SamplerWidget::SamplerWidget(TrackManager& track_manager) : m_track_manager(track_manager) { set_layout(); - layout()->set_margins({ 10, 10, 10, 10 }); + layout()->set_margins(10); layout()->set_spacing(10); set_fill_with_background_color(true); diff --git a/Userland/Applications/PixelPaint/CreateNewGuideDialog.gml b/Userland/Applications/PixelPaint/CreateNewGuideDialog.gml index 54cf57a6d8..effd464e37 100644 --- a/Userland/Applications/PixelPaint/CreateNewGuideDialog.gml +++ b/Userland/Applications/PixelPaint/CreateNewGuideDialog.gml @@ -6,7 +6,7 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [4,4,4,4] + margins: [4] } @GUI::GroupBox { @@ -14,7 +14,7 @@ shrink_to_fit: true layout: @GUI::HorizontalBoxLayout { - margins: [20,10,10,10] + margins: [20, 10, 10] } @GUI::RadioButton { @@ -31,7 +31,7 @@ @GUI::Widget { layout: @GUI::HorizontalBoxLayout { - margins: [4,4,4,4] + margins: [4] } shrink_to_fit: true @@ -50,7 +50,7 @@ max_height: 24 layout: @GUI::HorizontalBoxLayout { - margins: [4,4,4,4] + margins: [4] } @GUI::Widget { diff --git a/Userland/Applications/PixelPaint/CreateNewImageDialog.cpp b/Userland/Applications/PixelPaint/CreateNewImageDialog.cpp index b525b1f44c..1a5dd5e057 100644 --- a/Userland/Applications/PixelPaint/CreateNewImageDialog.cpp +++ b/Userland/Applications/PixelPaint/CreateNewImageDialog.cpp @@ -23,7 +23,7 @@ CreateNewImageDialog::CreateNewImageDialog(GUI::Window* parent_window) main_widget.set_fill_with_background_color(true); auto& layout = main_widget.set_layout(); - layout.set_margins({ 4, 4, 4, 4 }); + layout.set_margins(4); auto& name_label = main_widget.add("Name:"); name_label.set_text_alignment(Gfx::TextAlignment::CenterLeft); diff --git a/Userland/Applications/PixelPaint/CreateNewLayerDialog.cpp b/Userland/Applications/PixelPaint/CreateNewLayerDialog.cpp index c0b46f8aae..14e76964e0 100644 --- a/Userland/Applications/PixelPaint/CreateNewLayerDialog.cpp +++ b/Userland/Applications/PixelPaint/CreateNewLayerDialog.cpp @@ -24,7 +24,7 @@ CreateNewLayerDialog::CreateNewLayerDialog(Gfx::IntSize const& suggested_size, G main_widget.set_fill_with_background_color(true); auto& layout = main_widget.set_layout(); - layout.set_margins({ 4, 4, 4, 4 }); + layout.set_margins(4); auto& name_label = main_widget.add("Name:"); name_label.set_text_alignment(Gfx::TextAlignment::CenterLeft); diff --git a/Userland/Applications/PixelPaint/FilterParams.h b/Userland/Applications/PixelPaint/FilterParams.h index 6861665d3a..906e7d6a42 100644 --- a/Userland/Applications/PixelPaint/FilterParams.h +++ b/Userland/Applications/PixelPaint/FilterParams.h @@ -50,7 +50,7 @@ private: main_widget.set_frame_shadow(Gfx::FrameShadow::Raised); main_widget.set_fill_with_background_color(true); auto& layout = main_widget.template set_layout(); - layout.set_margins({ 4, 4, 4, 4 }); + layout.set_margins(4); size_t index = 0; size_t columns = N; diff --git a/Userland/Applications/PixelPaint/LayerPropertiesWidget.cpp b/Userland/Applications/PixelPaint/LayerPropertiesWidget.cpp index 67094d6554..c0d1fbbb9d 100644 --- a/Userland/Applications/PixelPaint/LayerPropertiesWidget.cpp +++ b/Userland/Applications/PixelPaint/LayerPropertiesWidget.cpp @@ -25,7 +25,7 @@ LayerPropertiesWidget::LayerPropertiesWidget() auto& group_box = add("Layer properties"); auto& layout = group_box.set_layout(); - layout.set_margins({ 20, 10, 10, 10 }); + layout.set_margins({ 20, 10, 10 }); auto& name_container = group_box.add(); name_container.set_fixed_height(20); diff --git a/Userland/Applications/PixelPaint/PixelPaintWindow.gml b/Userland/Applications/PixelPaint/PixelPaintWindow.gml index c008f276f8..0936c650c5 100644 --- a/Userland/Applications/PixelPaint/PixelPaintWindow.gml +++ b/Userland/Applications/PixelPaint/PixelPaintWindow.gml @@ -51,7 +51,7 @@ @GUI::GroupBox { title: "Layers" layout: @GUI::VerticalBoxLayout { - margins: [16, 6, 6, 6] + margins: [16, 6, 6] } @PixelPaint::LayerListWidget { diff --git a/Userland/Applications/PixelPaint/ToolPropertiesWidget.cpp b/Userland/Applications/PixelPaint/ToolPropertiesWidget.cpp index 9e96db24b2..6aa245ea62 100644 --- a/Userland/Applications/PixelPaint/ToolPropertiesWidget.cpp +++ b/Userland/Applications/PixelPaint/ToolPropertiesWidget.cpp @@ -19,7 +19,7 @@ ToolPropertiesWidget::ToolPropertiesWidget() m_group_box = add("Tool properties"); auto& layout = m_group_box->set_layout(); - layout.set_margins({ 20, 10, 10, 10 }); + layout.set_margins({ 20, 10, 10 }); m_tool_widget_stack = m_group_box->add(); m_blank_widget = m_tool_widget_stack->add(); } diff --git a/Userland/Applications/PixelPaint/ToolboxWidget.cpp b/Userland/Applications/PixelPaint/ToolboxWidget.cpp index 55481348a2..b8c4323d7a 100644 --- a/Userland/Applications/PixelPaint/ToolboxWidget.cpp +++ b/Userland/Applications/PixelPaint/ToolboxWidget.cpp @@ -35,7 +35,7 @@ ToolboxWidget::ToolboxWidget() set_layout(); layout()->set_spacing(0); - layout()->set_margins({ 2, 2, 2, 2 }); + layout()->set_margins(2); m_action_group.set_exclusive(true); m_action_group.set_unchecking_allowed(false); diff --git a/Userland/Applications/Run/Run.gml b/Userland/Applications/Run/Run.gml index 53f2d94b9f..99030fb934 100644 --- a/Userland/Applications/Run/Run.gml +++ b/Userland/Applications/Run/Run.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::Widget { @@ -24,7 +24,7 @@ @GUI::Widget { layout: @GUI::HorizontalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::Label { diff --git a/Userland/Applications/Spreadsheet/CellTypeDialog.cpp b/Userland/Applications/Spreadsheet/CellTypeDialog.cpp index f50a7efad6..bf59ca5ff0 100644 --- a/Userland/Applications/Spreadsheet/CellTypeDialog.cpp +++ b/Userland/Applications/Spreadsheet/CellTypeDialog.cpp @@ -46,7 +46,7 @@ CellTypeDialog::CellTypeDialog(const Vector& positions, Sheet& sheet, resize(285, 360); auto& main_widget = set_main_widget(); - main_widget.set_layout().set_margins({ 4, 4, 4, 4 }); + main_widget.set_layout().set_margins(4); main_widget.set_fill_with_background_color(true); auto& tab_widget = main_widget.add(); @@ -130,7 +130,7 @@ void CellTypeDialog::setup_tabs(GUI::TabWidget& tabs, const Vector& po } auto& type_tab = tabs.add_tab("Type"); - type_tab.set_layout().set_margins({ 4, 4, 4, 4 }); + type_tab.set_layout().set_margins(4); { auto& left_side = type_tab.add(); left_side.set_layout(); @@ -192,14 +192,14 @@ void CellTypeDialog::setup_tabs(GUI::TabWidget& tabs, const Vector& po } auto& alignment_tab = tabs.add_tab("Alignment"); - alignment_tab.set_layout().set_margins({ 4, 4, 4, 4 }); + alignment_tab.set_layout().set_margins(4); { // FIXME: Frame? // Horizontal alignment { auto& horizontal_alignment_selection_container = alignment_tab.add(); horizontal_alignment_selection_container.set_layout(); - horizontal_alignment_selection_container.layout()->set_margins({ 4, 0, 0, 0 }); + horizontal_alignment_selection_container.layout()->set_margins({ 4, 0, 0 }); horizontal_alignment_selection_container.set_fixed_height(22); auto& horizontal_alignment_label = horizontal_alignment_selection_container.add(); @@ -231,7 +231,7 @@ void CellTypeDialog::setup_tabs(GUI::TabWidget& tabs, const Vector& po { auto& vertical_alignment_container = alignment_tab.add(); vertical_alignment_container.set_layout(); - vertical_alignment_container.layout()->set_margins({ 0, 4, 0, 0 }); + vertical_alignment_container.layout()->set_margins({ 4, 0, 0 }); vertical_alignment_container.set_fixed_height(22); auto& vertical_alignment_label = vertical_alignment_container.add(); @@ -261,7 +261,7 @@ void CellTypeDialog::setup_tabs(GUI::TabWidget& tabs, const Vector& po } auto& colors_tab = tabs.add_tab("Color"); - colors_tab.set_layout().set_margins({ 4, 4, 4, 4 }); + colors_tab.set_layout().set_margins(4); { // Static formatting { @@ -273,7 +273,7 @@ void CellTypeDialog::setup_tabs(GUI::TabWidget& tabs, const Vector& po // FIXME: Somehow allow unsetting these. auto& foreground_container = static_formatting_container.add(); foreground_container.set_layout(); - foreground_container.layout()->set_margins({ 4, 0, 0, 0 }); + foreground_container.layout()->set_margins({ 4, 0, 0 }); foreground_container.set_shrink_to_fit(true); auto& foreground_label = foreground_container.add(); @@ -293,7 +293,7 @@ void CellTypeDialog::setup_tabs(GUI::TabWidget& tabs, const Vector& po // FIXME: Somehow allow unsetting these. auto& background_container = static_formatting_container.add(); background_container.set_layout(); - background_container.layout()->set_margins({ 4, 0, 0, 0 }); + background_container.layout()->set_margins({ 4, 0, 0 }); background_container.set_shrink_to_fit(true); auto& background_label = background_container.add(); diff --git a/Userland/Applications/Spreadsheet/CondFormatting.gml b/Userland/Applications/Spreadsheet/CondFormatting.gml index 79617f2894..9157968007 100644 --- a/Userland/Applications/Spreadsheet/CondFormatting.gml +++ b/Userland/Applications/Spreadsheet/CondFormatting.gml @@ -3,7 +3,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] spacing: 4 } diff --git a/Userland/Applications/Spreadsheet/SpreadsheetView.cpp b/Userland/Applications/Spreadsheet/SpreadsheetView.cpp index bd3a451911..ef2c6ec1ab 100644 --- a/Userland/Applications/Spreadsheet/SpreadsheetView.cpp +++ b/Userland/Applications/Spreadsheet/SpreadsheetView.cpp @@ -151,7 +151,7 @@ SpreadsheetView::SpreadsheetView(Sheet& sheet) : m_sheet(sheet) , m_sheet_model(SheetModel::create(*m_sheet)) { - set_layout().set_margins({ 2, 2, 2, 2 }); + set_layout().set_margins(2); m_table_view = add(); m_table_view->set_grid_style(GUI::TableView::GridStyle::Both); m_table_view->set_selection_behavior(GUI::AbstractView::SelectionBehavior::SelectItems); diff --git a/Userland/Applications/Spreadsheet/SpreadsheetWidget.cpp b/Userland/Applications/Spreadsheet/SpreadsheetWidget.cpp index f7c5a64c3d..5d2abc435e 100644 --- a/Userland/Applications/Spreadsheet/SpreadsheetWidget.cpp +++ b/Userland/Applications/Spreadsheet/SpreadsheetWidget.cpp @@ -26,7 +26,7 @@ SpreadsheetWidget::SpreadsheetWidget(NonnullRefPtrVector&& sheets, bool s : m_workbook(make(move(sheets))) { set_fill_with_background_color(true); - set_layout().set_margins({ 2, 2, 2, 2 }); + set_layout().set_margins(2); auto& container = add(); auto& top_bar = container.add(); @@ -71,7 +71,7 @@ SpreadsheetWidget::SpreadsheetWidget(NonnullRefPtrVector&& sheets, bool s m_inline_documentation_window->set_resizable(false); auto& inline_widget = m_inline_documentation_window->set_main_widget(); inline_widget.set_fill_with_background_color(true); - inline_widget.set_layout().set_margins({ 4, 4, 4, 4 }); + inline_widget.set_layout().set_margins(4); inline_widget.set_frame_shape(Gfx::FrameShape::Box); m_inline_documentation_label = inline_widget.add(); m_inline_documentation_label->set_fill_with_background_color(true); diff --git a/Userland/Applications/Spreadsheet/csv_export.gml b/Userland/Applications/Spreadsheet/csv_export.gml index e07706b207..4f0fa92ce0 100644 --- a/Userland/Applications/Spreadsheet/csv_export.gml +++ b/Userland/Applications/Spreadsheet/csv_export.gml @@ -1,6 +1,6 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [20, 20, 20, 20] + margins: [20] } @GUI::HorizontalSplitter { @@ -19,7 +19,7 @@ layout: @GUI::VerticalBoxLayout { // FIXME: This is working around the fact that group boxes don't allocate space for their title and border! - margins: [20, 10, 10, 10] + margins: [20, 10, 10] } @GUI::RadioButton { @@ -70,7 +70,7 @@ layout: @GUI::VerticalBoxLayout { // FIXME: This is working around the fact that group boxes don't allocate space for their title and border! - margins: [20, 10, 10, 10] + margins: [20, 10, 10] } @GUI::RadioButton { @@ -152,7 +152,7 @@ layout: @GUI::VerticalBoxLayout { // FIXME: This is working around the fact that group boxes don't allocate space for their title and border! - margins: [20, 10, 10, 10] + margins: [20, 10, 10] } @GUI::TextEditor { diff --git a/Userland/Applications/Spreadsheet/csv_import.gml b/Userland/Applications/Spreadsheet/csv_import.gml index eb141dcf40..80e6a99e2d 100644 --- a/Userland/Applications/Spreadsheet/csv_import.gml +++ b/Userland/Applications/Spreadsheet/csv_import.gml @@ -1,6 +1,6 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [20, 20, 20, 20] + margins: [20] } @GUI::HorizontalSplitter { @@ -19,7 +19,7 @@ layout: @GUI::VerticalBoxLayout { // FIXME: This is working around the fact that group boxes don't allocate space for their title and border! - margins: [20, 10, 10, 10] + margins: [20, 10, 10] } @GUI::RadioButton { @@ -70,7 +70,7 @@ layout: @GUI::VerticalBoxLayout { // FIXME: This is working around the fact that group boxes don't allocate space for their title and border! - margins: [20, 10, 10, 10] + margins: [20, 10, 10] } @GUI::RadioButton { @@ -133,7 +133,7 @@ fixed_height: 40 layout: @GUI::VerticalBoxLayout { - margins: [6, 6, 0, 6] + margins: [6, 6, 0] } @GUI::Widget { @@ -165,7 +165,7 @@ layout: @GUI::VerticalBoxLayout { // FIXME: This is working around the fact that group boxes don't allocate space for their title and border! - margins: [20, 10, 10, 10] + margins: [20, 10, 10] } @GUI::StackWidget { diff --git a/Userland/Applications/Spreadsheet/select_format_page.gml b/Userland/Applications/Spreadsheet/select_format_page.gml index 66b99025f3..7010ce8e38 100644 --- a/Userland/Applications/Spreadsheet/select_format_page.gml +++ b/Userland/Applications/Spreadsheet/select_format_page.gml @@ -2,7 +2,7 @@ name: "select_format" layout: @GUI::VerticalBoxLayout { - margins: [20, 20, 20, 20] + margins: [20] } @GUI::Label { diff --git a/Userland/Applications/SystemMonitor/InterruptsWidget.cpp b/Userland/Applications/SystemMonitor/InterruptsWidget.cpp index 594ad29da9..f46f01590b 100644 --- a/Userland/Applications/SystemMonitor/InterruptsWidget.cpp +++ b/Userland/Applications/SystemMonitor/InterruptsWidget.cpp @@ -15,7 +15,7 @@ InterruptsWidget::InterruptsWidget() { on_first_show = [this](auto&) { set_layout(); - layout()->set_margins({ 4, 4, 4, 4 }); + layout()->set_margins(4); Vector interrupts_field; interrupts_field.empend("interrupt_line", "Line", Gfx::TextAlignment::CenterRight); diff --git a/Userland/Applications/SystemMonitor/MemoryStatsWidget.cpp b/Userland/Applications/SystemMonitor/MemoryStatsWidget.cpp index 4dcedb3340..3e435553e3 100644 --- a/Userland/Applications/SystemMonitor/MemoryStatsWidget.cpp +++ b/Userland/Applications/SystemMonitor/MemoryStatsWidget.cpp @@ -31,7 +31,7 @@ MemoryStatsWidget::MemoryStatsWidget(GraphWidget& graph) set_fixed_height(110); set_layout(); - layout()->set_margins({ 8, 0, 0, 0 }); + layout()->set_margins({ 8, 0, 0 }); layout()->set_spacing(3); auto build_widgets_for_label = [this](const String& description) -> RefPtr { diff --git a/Userland/Applications/SystemMonitor/NetworkStatisticsWidget.cpp b/Userland/Applications/SystemMonitor/NetworkStatisticsWidget.cpp index 77383c831d..604e16b18d 100644 --- a/Userland/Applications/SystemMonitor/NetworkStatisticsWidget.cpp +++ b/Userland/Applications/SystemMonitor/NetworkStatisticsWidget.cpp @@ -16,7 +16,7 @@ NetworkStatisticsWidget::NetworkStatisticsWidget() { on_first_show = [this](auto&) { set_layout(); - layout()->set_margins({ 4, 4, 4, 4 }); + layout()->set_margins(4); set_fill_with_background_color(true); m_network_connected_bitmap = Gfx::Bitmap::try_load_from_file("/res/icons/16x16/network-connected.png"); @@ -32,7 +32,7 @@ NetworkStatisticsWidget::NetworkStatisticsWidget() auto& adapters_group_box = add("Adapters"); adapters_group_box.set_layout(); - adapters_group_box.layout()->set_margins({ 16, 6, 6, 6 }); + adapters_group_box.layout()->set_margins({ 16, 6, 6 }); adapters_group_box.set_fixed_height(120); m_adapter_table_view = adapters_group_box.add(); @@ -69,7 +69,7 @@ NetworkStatisticsWidget::NetworkStatisticsWidget() auto& tcp_sockets_group_box = add("TCP Sockets"); tcp_sockets_group_box.set_layout(); - tcp_sockets_group_box.layout()->set_margins({ 16, 6, 6, 6 }); + tcp_sockets_group_box.layout()->set_margins({ 16, 6, 6 }); m_tcp_socket_table_view = tcp_sockets_group_box.add(); @@ -90,7 +90,7 @@ NetworkStatisticsWidget::NetworkStatisticsWidget() auto& udp_sockets_group_box = add("UDP Sockets"); udp_sockets_group_box.set_layout(); - udp_sockets_group_box.layout()->set_margins({ 16, 6, 6, 6 }); + udp_sockets_group_box.layout()->set_margins({ 16, 6, 6 }); m_udp_socket_table_view = udp_sockets_group_box.add(); diff --git a/Userland/Applications/SystemMonitor/ProcessFileDescriptorMapWidget.cpp b/Userland/Applications/SystemMonitor/ProcessFileDescriptorMapWidget.cpp index 8a4f3a0e38..879983aedf 100644 --- a/Userland/Applications/SystemMonitor/ProcessFileDescriptorMapWidget.cpp +++ b/Userland/Applications/SystemMonitor/ProcessFileDescriptorMapWidget.cpp @@ -13,7 +13,7 @@ ProcessFileDescriptorMapWidget::ProcessFileDescriptorMapWidget() { set_layout(); - layout()->set_margins({ 4, 4, 4, 4 }); + layout()->set_margins(4); m_table_view = add(); Vector pid_fds_fields; diff --git a/Userland/Applications/SystemMonitor/ProcessMemoryMapWidget.cpp b/Userland/Applications/SystemMonitor/ProcessMemoryMapWidget.cpp index ef4c156feb..6cf99fed78 100644 --- a/Userland/Applications/SystemMonitor/ProcessMemoryMapWidget.cpp +++ b/Userland/Applications/SystemMonitor/ProcessMemoryMapWidget.cpp @@ -47,7 +47,7 @@ public: ProcessMemoryMapWidget::ProcessMemoryMapWidget() { set_layout(); - layout()->set_margins({ 4, 4, 4, 4 }); + layout()->set_margins(4); m_table_view = add(); Vector pid_vm_fields; pid_vm_fields.empend( diff --git a/Userland/Applications/SystemMonitor/ProcessStateWidget.cpp b/Userland/Applications/SystemMonitor/ProcessStateWidget.cpp index 05e6c4cadb..bfe9706b75 100644 --- a/Userland/Applications/SystemMonitor/ProcessStateWidget.cpp +++ b/Userland/Applications/SystemMonitor/ProcessStateWidget.cpp @@ -84,7 +84,7 @@ private: ProcessStateWidget::ProcessStateWidget(pid_t pid) { set_layout(); - layout()->set_margins({ 4, 4, 4, 4 }); + layout()->set_margins(4); m_table_view = add(); m_table_view->set_model(adopt_ref(*new ProcessStateModel(ProcessModel::the(), pid))); m_table_view->column_header().set_visible(false); diff --git a/Userland/Applications/SystemMonitor/ProcessUnveiledPathsWidget.cpp b/Userland/Applications/SystemMonitor/ProcessUnveiledPathsWidget.cpp index 26ba36e1a9..2ad0045bac 100644 --- a/Userland/Applications/SystemMonitor/ProcessUnveiledPathsWidget.cpp +++ b/Userland/Applications/SystemMonitor/ProcessUnveiledPathsWidget.cpp @@ -13,7 +13,7 @@ ProcessUnveiledPathsWidget::ProcessUnveiledPathsWidget() { set_layout(); - layout()->set_margins({ 4, 4, 4, 4 }); + layout()->set_margins(4); m_table_view = add(); Vector pid_unveil_fields; diff --git a/Userland/Applications/SystemMonitor/ThreadStackWidget.cpp b/Userland/Applications/SystemMonitor/ThreadStackWidget.cpp index a108faa749..d0ccef72a7 100644 --- a/Userland/Applications/SystemMonitor/ThreadStackWidget.cpp +++ b/Userland/Applications/SystemMonitor/ThreadStackWidget.cpp @@ -14,7 +14,7 @@ ThreadStackWidget::ThreadStackWidget() { set_layout(); - layout()->set_margins({ 4, 4, 4, 4 }); + layout()->set_margins(4); m_stack_editor = add(); m_stack_editor->set_mode(GUI::TextEditor::ReadOnly); m_stack_editor->set_text("Symbolicating..."); diff --git a/Userland/Applications/SystemMonitor/main.cpp b/Userland/Applications/SystemMonitor/main.cpp index 496bd4d4a8..795b34d028 100644 --- a/Userland/Applications/SystemMonitor/main.cpp +++ b/Userland/Applications/SystemMonitor/main.cpp @@ -179,7 +179,7 @@ int main(int argc, char** argv) auto& tabwidget_container = main_widget.add(); tabwidget_container.set_layout(); - tabwidget_container.layout()->set_margins({ 0, 4, 4, 4 }); + tabwidget_container.layout()->set_margins({ 0, 4, 4 }); auto& tabwidget = tabwidget_container.add(); statusbar = main_widget.add(3); @@ -214,7 +214,7 @@ int main(int argc, char** argv) tabwidget.add_widget("Interrupts", interrupts_widget); process_table_container.set_layout(); - process_table_container.layout()->set_margins({ 4, 4, 4, 4 }); + process_table_container.layout()->set_margins(4); process_table_container.layout()->set_spacing(0); auto& process_table_view = process_table_container.add(); @@ -434,7 +434,7 @@ NonnullRefPtr build_process_window(pid_t pid) auto& hero_container = main_widget.add(); hero_container.set_shrink_to_fit(true); hero_container.set_layout(); - hero_container.layout()->set_margins({ 4, 4, 4, 4 }); + hero_container.layout()->set_margins(4); hero_container.layout()->set_spacing(8); auto& icon_label = hero_container.add(); @@ -493,7 +493,7 @@ NonnullRefPtr build_file_systems_tab() fs_widget->on_first_show = [](GUI::LazyWidget& self) { self.set_layout(); - self.layout()->set_margins({ 4, 4, 4, 4 }); + self.layout()->set_margins(4); auto& fs_table_view = self.add(); Vector df_fields; @@ -589,7 +589,7 @@ NonnullRefPtr build_pci_devices_tab() pci_widget->on_first_show = [](GUI::LazyWidget& self) { self.set_layout(); - self.layout()->set_margins({ 4, 4, 4, 4 }); + self.layout()->set_margins(4); auto& pci_table_view = self.add(); auto db = PCIDB::Database::open(); @@ -648,7 +648,7 @@ NonnullRefPtr build_devices_tab() devices_widget->on_first_show = [](GUI::LazyWidget& self) { self.set_layout(); - self.layout()->set_margins({ 4, 4, 4, 4 }); + self.layout()->set_margins(4); auto& devices_table_view = self.add(); devices_table_view.set_model(GUI::SortingProxyModel::create(DevicesModel::create())); @@ -665,11 +665,11 @@ NonnullRefPtr build_graphs_tab() graphs_container->set_fill_with_background_color(true); graphs_container->set_background_role(ColorRole::Button); graphs_container->set_layout(); - graphs_container->layout()->set_margins({ 4, 4, 4, 4 }); + graphs_container->layout()->set_margins(4); auto& cpu_graph_group_box = graphs_container->add("CPU usage"); cpu_graph_group_box.set_layout(); - cpu_graph_group_box.layout()->set_margins({ 16, 6, 6, 6 }); + cpu_graph_group_box.layout()->set_margins({ 16, 6, 6 }); cpu_graph_group_box.set_fixed_height(120); Vector cpu_graphs; for (size_t i = 0; i < ProcessModel::the().cpus().size(); i++) { @@ -701,7 +701,7 @@ NonnullRefPtr build_graphs_tab() auto& memory_graph_group_box = graphs_container->add("Memory usage"); memory_graph_group_box.set_layout(); - memory_graph_group_box.layout()->set_margins({ 16, 6, 6, 6 }); + memory_graph_group_box.layout()->set_margins({ 16, 6, 6 }); memory_graph_group_box.set_fixed_height(120); auto& memory_graph = memory_graph_group_box.add(); memory_graph.set_stack_values(true); @@ -734,7 +734,7 @@ NonnullRefPtr build_processors_tab() processors_widget->on_first_show = [](GUI::LazyWidget& self) { self.set_layout(); - self.layout()->set_margins({ 4, 4, 4, 4 }); + self.layout()->set_margins(4); Vector processors_field; processors_field.empend("processor", "Processor", Gfx::TextAlignment::CenterRight); diff --git a/Userland/Applications/Terminal/TerminalSettingsWindow.gml b/Userland/Applications/Terminal/TerminalSettingsWindow.gml index 575257215b..33718a2e0b 100644 --- a/Userland/Applications/Terminal/TerminalSettingsWindow.gml +++ b/Userland/Applications/Terminal/TerminalSettingsWindow.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::GroupBox { @@ -10,7 +10,7 @@ shrink_to_fit: true layout: @GUI::VerticalBoxLayout { - margins: [16, 6, 6, 6] + margins: [16, 6, 6] } @GUI::RadioButton { @@ -34,7 +34,7 @@ shrink_to_fit: true layout: @GUI::VerticalBoxLayout { - margins: [16, 6, 6, 6] + margins: [16, 6, 6] } @GUI::OpacitySlider { @@ -50,7 +50,7 @@ shrink_to_fit: true layout: @GUI::VerticalBoxLayout { - margins: [16, 6, 6, 6] + margins: [16, 6, 6] } @GUI::SpinBox { @@ -66,7 +66,7 @@ shrink_to_fit: true layout: @GUI::VerticalBoxLayout { - margins: [16, 6, 6, 6] + margins: [16, 6, 6] } @GUI::ComboBox { diff --git a/Userland/Applications/Terminal/main.cpp b/Userland/Applications/Terminal/main.cpp index 4840e319c0..dd67defb99 100644 --- a/Userland/Applications/Terminal/main.cpp +++ b/Userland/Applications/Terminal/main.cpp @@ -190,11 +190,11 @@ static RefPtr create_find_window(VT::TerminalWidget& terminal) search.set_fill_with_background_color(true); search.set_background_role(ColorRole::Button); search.set_layout(); - search.layout()->set_margins({ 4, 4, 4, 4 }); + search.layout()->set_margins(4); auto& find = search.add(); find.set_layout(); - find.layout()->set_margins({ 4, 4, 4, 4 }); + find.layout()->set_margins(4); find.set_fixed_height(30); auto& find_textbox = find.add(); diff --git a/Userland/Applications/TextEditor/TextEditorWindow.gml b/Userland/Applications/TextEditor/TextEditorWindow.gml index 3d7b06f1be..015112eccc 100644 --- a/Userland/Applications/TextEditor/TextEditorWindow.gml +++ b/Userland/Applications/TextEditor/TextEditorWindow.gml @@ -36,7 +36,7 @@ layout: @GUI::VerticalBoxLayout { spacing: 2 - margins: [5, 5, 5, 5] + margins: [5] } @GUI::Widget { diff --git a/Userland/Demos/WidgetGallery/DemoWizardPage1.gml b/Userland/Demos/WidgetGallery/DemoWizardPage1.gml index 136ff9296c..047eb38fbf 100644 --- a/Userland/Demos/WidgetGallery/DemoWizardPage1.gml +++ b/Userland/Demos/WidgetGallery/DemoWizardPage1.gml @@ -1,6 +1,6 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [20, 20, 20, 20] + margins: [20] } @GUI::Label { diff --git a/Userland/Demos/WidgetGallery/DemoWizardPage2.gml b/Userland/Demos/WidgetGallery/DemoWizardPage2.gml index 262d25864d..6f6d6b7f25 100644 --- a/Userland/Demos/WidgetGallery/DemoWizardPage2.gml +++ b/Userland/Demos/WidgetGallery/DemoWizardPage2.gml @@ -1,6 +1,6 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [20, 20, 20, 20] + margins: [20] } @GUI::Label { diff --git a/Userland/Demos/WidgetGallery/GalleryGML/BasicsTab.gml b/Userland/Demos/WidgetGallery/GalleryGML/BasicsTab.gml index 77ead7ffb5..b9028e3de7 100644 --- a/Userland/Demos/WidgetGallery/GalleryGML/BasicsTab.gml +++ b/Userland/Demos/WidgetGallery/GalleryGML/BasicsTab.gml @@ -1,13 +1,13 @@ @GUI::Widget { name: "basics_tab" layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::GroupBox { fixed_height: 95 layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::HorizontalSplitter { @@ -20,7 +20,7 @@ shadow: "Sunken" thickness: 1 layout: @GUI::VerticalBoxLayout { - margins: [3, 4, 3, 4] + margins: [3, 4] } @GUI::Label { @@ -37,7 +37,7 @@ @GUI::Widget { layout: @GUI::VerticalBoxLayout { - margins: [0, 4, 0, 4] + margins: [0, 4] } @GUI::Label { @@ -85,7 +85,7 @@ @GUI::Widget { fixed_height: 125 layout: @GUI::VerticalBoxLayout { - margins: [3, 8, 3, 8] + margins: [3, 8] } @GUI::Widget { @@ -237,7 +237,7 @@ @GUI::GroupBox { layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::Widget { diff --git a/Userland/Demos/WidgetGallery/GalleryGML/CursorsTab.gml b/Userland/Demos/WidgetGallery/GalleryGML/CursorsTab.gml index 965414eb10..581cf2dac1 100644 --- a/Userland/Demos/WidgetGallery/GalleryGML/CursorsTab.gml +++ b/Userland/Demos/WidgetGallery/GalleryGML/CursorsTab.gml @@ -1,12 +1,12 @@ @GUI::Widget { name: "cursors_tab" layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::GroupBox { layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::TableView { diff --git a/Userland/Demos/WidgetGallery/GalleryGML/IconsTab.gml b/Userland/Demos/WidgetGallery/GalleryGML/IconsTab.gml index 8a3496c799..02e5cd8b20 100644 --- a/Userland/Demos/WidgetGallery/GalleryGML/IconsTab.gml +++ b/Userland/Demos/WidgetGallery/GalleryGML/IconsTab.gml @@ -1,12 +1,12 @@ @GUI::Widget { name: "icons_tab" layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::GroupBox { layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::TableView { diff --git a/Userland/Demos/WidgetGallery/GalleryGML/SlidersTab.gml b/Userland/Demos/WidgetGallery/GalleryGML/SlidersTab.gml index cd02de5986..d2a5e88a78 100644 --- a/Userland/Demos/WidgetGallery/GalleryGML/SlidersTab.gml +++ b/Userland/Demos/WidgetGallery/GalleryGML/SlidersTab.gml @@ -1,13 +1,13 @@ @GUI::Widget { name: "sliders_tab" layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::GroupBox { fixed_height: 129 layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::OpacitySlider { @@ -24,7 +24,7 @@ max_width: 394 max_height: 79 layout: @GUI::VerticalBoxLayout { - margins: [1, 1, 1, 1] + margins: [1] } @GUI::ImageWidget { @@ -36,7 +36,7 @@ @GUI::Widget { fixed_height: 88 layout: @GUI::VerticalBoxLayout { - margins: [0, 8, 0, 8] + margins: [0, 8] } @GUI::Widget { @@ -72,7 +72,7 @@ @GUI::GroupBox { layout: @GUI::HorizontalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::VerticalProgressbar { @@ -120,7 +120,7 @@ @GUI::GroupBox { layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::Widget { diff --git a/Userland/Demos/WidgetGallery/GalleryGML/Window.gml b/Userland/Demos/WidgetGallery/GalleryGML/Window.gml index 33fd586ad5..4e7eb0c49e 100644 --- a/Userland/Demos/WidgetGallery/GalleryGML/Window.gml +++ b/Userland/Demos/WidgetGallery/GalleryGML/Window.gml @@ -1,7 +1,7 @@ @GUI::Widget { fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::TabWidget { diff --git a/Userland/Demos/WidgetGallery/GalleryGML/WizardsTab.gml b/Userland/Demos/WidgetGallery/GalleryGML/WizardsTab.gml index 2466a748d0..5aece78f1c 100644 --- a/Userland/Demos/WidgetGallery/GalleryGML/WizardsTab.gml +++ b/Userland/Demos/WidgetGallery/GalleryGML/WizardsTab.gml @@ -1,12 +1,12 @@ @GUI::Widget { name: "wizards_tab" layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::GroupBox { layout: @GUI::VerticalBoxLayout { - margins: [8, 8, 8, 8] + margins: [8] } @GUI::Button { diff --git a/Userland/DevTools/HackStudio/Debugger/EvaluateExpressionDialog.cpp b/Userland/DevTools/HackStudio/Debugger/EvaluateExpressionDialog.cpp index 49adb190bf..9750129519 100644 --- a/Userland/DevTools/HackStudio/Debugger/EvaluateExpressionDialog.cpp +++ b/Userland/DevTools/HackStudio/Debugger/EvaluateExpressionDialog.cpp @@ -48,7 +48,7 @@ void EvaluateExpressionDialog::build(Window* parent_window) widget.set_layout(); widget.set_fill_with_background_color(true); - widget.layout()->set_margins({ 6, 6, 6, 6 }); + widget.layout()->set_margins(6); widget.layout()->set_spacing(6); m_text_editor = widget.add(); @@ -77,7 +77,7 @@ void EvaluateExpressionDialog::build(Window* parent_window) auto& button_container_inner = button_container_outer.add(); button_container_inner.set_layout(); button_container_inner.layout()->set_spacing(6); - button_container_inner.layout()->set_margins({ 4, 0, 4, 4 }); + button_container_inner.layout()->set_margins({ 4, 0, 4 }); button_container_inner.layout()->add_spacer(); m_evaluate_button = button_container_inner.add(); diff --git a/Userland/DevTools/HackStudio/Dialogs/NewProjectDialog.gml b/Userland/DevTools/HackStudio/Dialogs/NewProjectDialog.gml index 41a1703f22..5387b1d973 100644 --- a/Userland/DevTools/HackStudio/Dialogs/NewProjectDialog.gml +++ b/Userland/DevTools/HackStudio/Dialogs/NewProjectDialog.gml @@ -2,7 +2,7 @@ fill_with_background_color: true layout: @GUI::VerticalBoxLayout { - margins: [4, 4, 4, 4] + margins: [4] } @GUI::Label { diff --git a/Userland/DevTools/HackStudio/EditorWrapper.cpp b/Userland/DevTools/HackStudio/EditorWrapper.cpp index 924be3a92f..73f7588998 100644 --- a/Userland/DevTools/HackStudio/EditorWrapper.cpp +++ b/Userland/DevTools/HackStudio/EditorWrapper.cpp @@ -24,7 +24,7 @@ EditorWrapper::EditorWrapper() label_wrapper.set_fixed_height(14); label_wrapper.set_fill_with_background_color(true); label_wrapper.set_layout(); - label_wrapper.layout()->set_margins({ 0, 2, 0, 2 }); + label_wrapper.layout()->set_margins({ 0, 2 }); m_filename_label = label_wrapper.add("(Untitled)"); m_filename_label->set_text_alignment(Gfx::TextAlignment::CenterLeft); diff --git a/Userland/DevTools/HackStudio/HackStudioWidget.cpp b/Userland/DevTools/HackStudio/HackStudioWidget.cpp index d77bc7c207..15db0dab88 100644 --- a/Userland/DevTools/HackStudio/HackStudioWidget.cpp +++ b/Userland/DevTools/HackStudio/HackStudioWidget.cpp @@ -107,7 +107,7 @@ HackStudioWidget::HackStudioWidget(const String& path_to_project) m_editors_splitter = m_right_hand_stack->add(); m_editors_splitter->layout()->set_spacing(5); - m_editors_splitter->layout()->set_margins({ 3, 0, 0, 0 }); + m_editors_splitter->layout()->set_margins({ 3, 0, 0 }); add_new_editor(*m_editors_splitter); m_switch_to_next_editor = create_switch_to_next_editor_action(); @@ -958,14 +958,14 @@ void HackStudioWidget::create_project_tab(GUI::Widget& parent) auto& tree_view_container = m_project_tab->add_tab("Files"); tree_view_container.set_layout(); - tree_view_container.layout()->set_margins({ 2, 2, 2, 2 }); + tree_view_container.layout()->set_margins(2); m_project_tree_view = tree_view_container.add(); configure_project_tree_view(); auto& class_view_container = m_project_tab->add_tab("Classes"); class_view_container.set_layout(); - class_view_container.layout()->set_margins({ 2, 2, 2, 2 }); + class_view_container.layout()->set_margins(2); m_class_view = class_view_container.add(); diff --git a/Userland/DevTools/Profiler/main.cpp b/Userland/DevTools/Profiler/main.cpp index b563541be6..344fac6350 100644 --- a/Userland/DevTools/Profiler/main.cpp +++ b/Userland/DevTools/Profiler/main.cpp @@ -131,7 +131,7 @@ int main(int argc, char** argv) auto& tree_tab = tab_widget.add_tab("Call Tree"); tree_tab.set_layout(); - tree_tab.layout()->set_margins({ 4, 4, 4, 4 }); + tree_tab.layout()->set_margins(4); auto& bottom_splitter = tree_tab.add(); auto& tree_view = bottom_splitter.add(); @@ -163,7 +163,7 @@ int main(int argc, char** argv) auto& samples_tab = tab_widget.add_tab("Samples"); samples_tab.set_layout(); - samples_tab.layout()->set_margins({ 4, 4, 4, 4 }); + samples_tab.layout()->set_margins(4); auto& samples_splitter = samples_tab.add(); auto& samples_table_view = samples_splitter.add(); @@ -178,7 +178,7 @@ int main(int argc, char** argv) auto& signposts_tab = tab_widget.add_tab("Signposts"); signposts_tab.set_layout(); - signposts_tab.layout()->set_margins({ 4, 4, 4, 4 }); + signposts_tab.layout()->set_margins(4); auto& signposts_splitter = signposts_tab.add(); auto& signposts_table_view = signposts_splitter.add(); @@ -262,7 +262,7 @@ static bool prompt_to_stop_profiling(pid_t pid, const String& process_name) auto& widget = window->set_main_widget(); widget.set_fill_with_background_color(true); auto& layout = widget.set_layout(); - layout.set_margins({ 0, 0, 16, 0 }); + layout.set_margins({ 0, 0, 16 }); auto& timer_label = widget.add("..."); Core::ElapsedTimer clock; diff --git a/Userland/Games/2048/GameSizeDialog.cpp b/Userland/Games/2048/GameSizeDialog.cpp index 0e1ca13312..15d80e7571 100644 --- a/Userland/Games/2048/GameSizeDialog.cpp +++ b/Userland/Games/2048/GameSizeDialog.cpp @@ -28,7 +28,7 @@ GameSizeDialog::GameSizeDialog(GUI::Window* parent, size_t board_size, size_t ta main_widget.set_fill_with_background_color(true); auto& layout = main_widget.set_layout(); - layout.set_margins({ 4, 4, 4, 4 }); + layout.set_margins(4); auto& board_size_box = main_widget.add(); auto& input_layout = board_size_box.set_layout(); diff --git a/Userland/Games/Breakout/LevelSelectDialog.cpp b/Userland/Games/Breakout/LevelSelectDialog.cpp index e420b2657f..e3ae74676b 100644 --- a/Userland/Games/Breakout/LevelSelectDialog.cpp +++ b/Userland/Games/Breakout/LevelSelectDialog.cpp @@ -41,7 +41,7 @@ void LevelSelectDialog::build() main_widget.set_fill_with_background_color(true); auto& layout = main_widget.set_layout(); - layout.set_margins({ 4, 4, 4, 4 }); + layout.set_margins(4); main_widget.add("Choose a level").set_text_alignment(Gfx::TextAlignment::Center); diff --git a/Userland/Games/GameOfLife/main.cpp b/Userland/Games/GameOfLife/main.cpp index 9bc8705f11..30f627fb76 100644 --- a/Userland/Games/GameOfLife/main.cpp +++ b/Userland/Games/GameOfLife/main.cpp @@ -40,7 +40,7 @@ int main(int argc, char** argv) main_widget.set_fill_with_background_color(true); auto& main_toolbar = *main_widget.find_descendant_of_type_named("toolbar"); - main_toolbar.layout()->set_margins({ 0, 6, 0, 6 }); + main_toolbar.layout()->set_margins({ 0, 6 }); auto& board_widget_container = *main_widget.find_descendant_of_type_named("board_widget_container"); auto& board_layout = board_widget_container.set_layout(); diff --git a/Userland/Games/Hearts/Game.cpp b/Userland/Games/Hearts/Game.cpp index cb023b95a9..2cc472715f 100644 --- a/Userland/Games/Hearts/Game.cpp +++ b/Userland/Games/Hearts/Game.cpp @@ -128,7 +128,7 @@ void Game::show_score_card(bool game_over) auto& score_widget = score_dialog->set_main_widget(); score_widget.set_fill_with_background_color(true); auto& layout = score_widget.set_layout(); - layout.set_margins({ 10, 10, 10, 10 }); + layout.set_margins(10); layout.set_spacing(15); auto& card_container = score_widget.add(); diff --git a/Userland/Games/Hearts/SettingsDialog.cpp b/Userland/Games/Hearts/SettingsDialog.cpp index 12faeac4b5..2e027ad4f4 100644 --- a/Userland/Games/Hearts/SettingsDialog.cpp +++ b/Userland/Games/Hearts/SettingsDialog.cpp @@ -23,7 +23,7 @@ SettingsDialog::SettingsDialog(GUI::Window* parent, String player_name) main_widget.set_fill_with_background_color(true); auto& layout = main_widget.set_layout(); - layout.set_margins({ 4, 4, 4, 4 }); + layout.set_margins(4); auto& name_box = main_widget.add(); auto& input_layout = name_box.set_layout(); diff --git a/Userland/Libraries/LibGUI/AboutDialog.cpp b/Userland/Libraries/LibGUI/AboutDialog.cpp index ef2c855e5b..8af316bd0e 100644 --- a/Userland/Libraries/LibGUI/AboutDialog.cpp +++ b/Userland/Libraries/LibGUI/AboutDialog.cpp @@ -43,7 +43,7 @@ AboutDialog::AboutDialog(const StringView& name, const Gfx::Bitmap* icon, Window auto& left_container = content_container.add(); left_container.set_fixed_width(60); left_container.set_layout(); - left_container.layout()->set_margins({ 12, 0, 0, 0 }); + left_container.layout()->set_margins({ 12, 0, 0 }); if (icon) { auto& icon_wrapper = left_container.add(); @@ -62,7 +62,7 @@ AboutDialog::AboutDialog(const StringView& name, const Gfx::Bitmap* icon, Window auto& label = right_container.add