From fb690c97e82dbb4ea8d5595d1f7ca3243f851078 Mon Sep 17 00:00:00 2001
From: Alberto Iannaccone <a.iannaccone@arduino.cc>
Date: Thu, 14 Jul 2022 14:50:46 +0200
Subject: [PATCH] Fix settings dialog size (#1172)

* give an id to the settings dialog to grant higher priority to css rule to fix the max-width

* fix settings dialog height
---
 .../src/browser/dialogs/settings/settings-dialog.tsx     | 2 +-
 .../src/browser/style/settings-dialog.css                | 9 +++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/arduino-ide-extension/src/browser/dialogs/settings/settings-dialog.tsx b/arduino-ide-extension/src/browser/dialogs/settings/settings-dialog.tsx
index 900d02d1..7e27def2 100644
--- a/arduino-ide-extension/src/browser/dialogs/settings/settings-dialog.tsx
+++ b/arduino-ide-extension/src/browser/dialogs/settings/settings-dialog.tsx
@@ -63,7 +63,7 @@ export class SettingsDialog extends AbstractDialog<Promise<Settings>> {
     protected override readonly props: SettingsDialogProps
   ) {
     super(props);
-    this.node.classList.add('arduino-settings-dialog-container');
+    this.node.id = 'arduino-settings-dialog-container';
     this.contentNode.classList.add('arduino-settings-dialog');
     this.appendCloseButton(
       nls.localize('vscode/issueMainService/cancel', 'Cancel')
diff --git a/arduino-ide-extension/src/browser/style/settings-dialog.css b/arduino-ide-extension/src/browser/style/settings-dialog.css
index 0c0482c5..5d389016 100644
--- a/arduino-ide-extension/src/browser/style/settings-dialog.css
+++ b/arduino-ide-extension/src/browser/style/settings-dialog.css
@@ -1,6 +1,11 @@
-.arduino-settings-dialog-container > .dialogBlock {
-    width: calc(100% - 96px);
+#arduino-settings-dialog-container > .dialogBlock {
+    height: 531px;;
     max-width: 740px !important;
+    width: calc(100% - 96px);
+}
+
+#arduino-settings-dialog-container > .dialogBlock > .dialogContent {
+    justify-content: flex-start;
 }
 
 .arduino-settings-dialog .content {