From 307d113594dd7baa6f08ca4b58bf9f181f71fc08 Mon Sep 17 00:00:00 2001
From: FrHun <28605587+frhun@users.noreply.github.com>
Date: Tue, 10 May 2022 23:33:11 +0200
Subject: [PATCH] Spreadsheet: Make conditional-formatting condition-list
 scrollable

---
 Userland/Applications/Spreadsheet/CellTypeDialog.cpp | 4 +++-
 Userland/Applications/Spreadsheet/CondFormatting.gml | 7 +++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/Userland/Applications/Spreadsheet/CellTypeDialog.cpp b/Userland/Applications/Spreadsheet/CellTypeDialog.cpp
index 1d70e5eca4d..c3d6a227792 100644
--- a/Userland/Applications/Spreadsheet/CellTypeDialog.cpp
+++ b/Userland/Applications/Spreadsheet/CellTypeDialog.cpp
@@ -427,7 +427,9 @@ ConditionView::~ConditionView()
 
 ConditionsView::ConditionsView()
 {
-    set_layout<GUI::VerticalBoxLayout>().set_spacing(2);
+    auto& layout = set_layout<GUI::VerticalBoxLayout>();
+    layout.set_spacing(4);
+    layout.set_margins({ 6 });
 }
 
 void ConditionsView::set_formats(Vector<ConditionalFormat>* formats)
diff --git a/Userland/Applications/Spreadsheet/CondFormatting.gml b/Userland/Applications/Spreadsheet/CondFormatting.gml
index 62c8a2e2490..59ca4f004e4 100644
--- a/Userland/Applications/Spreadsheet/CondFormatting.gml
+++ b/Userland/Applications/Spreadsheet/CondFormatting.gml
@@ -6,8 +6,11 @@
         spacing: 4
     }
 
-    @Spreadsheet::ConditionsView {
-        name: "conditions_view"
+    @GUI::ScrollableContainerWidget {
+        should_hide_unnecessary_scrollbars: true
+        content_widget: @Spreadsheet::ConditionsView {
+            name: "conditions_view"
+        }
     }
 
     @GUI::Widget {