From d8bdf269175c19ce3c9397d62239edda37e7c604 Mon Sep 17 00:00:00 2001
From: Karol Kosek <krkk@serenityos.org>
Date: Tue, 11 Jul 2023 22:41:47 +0200
Subject: [PATCH] MailSettings: Make TLS option checked by default

The window by default showed port 993 with TLS unchecked, which was
slightly misleading as port 993 uses implicit TLS connections and
cleartext is on port 143.

This change also makes it consistent with the Mail app, as its fallback
value is also set to true.
---
 Userland/Applications/MailSettings/MailSettingsWidget.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Userland/Applications/MailSettings/MailSettingsWidget.cpp b/Userland/Applications/MailSettings/MailSettingsWidget.cpp
index 822ff5d44c..5229d2484c 100644
--- a/Userland/Applications/MailSettings/MailSettingsWidget.cpp
+++ b/Userland/Applications/MailSettings/MailSettingsWidget.cpp
@@ -19,7 +19,7 @@ void MailSettingsWidget::reset_default_values()
 {
     m_server_inputbox->set_text(""sv);
     m_port_combobox->set_text("993"sv);
-    m_tls_checkbox->set_checked(false);
+    m_tls_checkbox->set_checked(true);
     m_email_inputbox->set_text(""sv);
 }
 
@@ -66,7 +66,7 @@ ErrorOr<void> MailSettingsWidget::setup()
     };
 
     m_tls_checkbox = *find_descendant_of_type_named<GUI::CheckBox>("tls_input");
-    m_tls_checkbox->set_checked(Config::read_bool("Mail"sv, "Connection"sv, "TLS"sv, false));
+    m_tls_checkbox->set_checked(Config::read_bool("Mail"sv, "Connection"sv, "TLS"sv, true));
     m_tls_checkbox->on_checked = [&](auto) {
         set_modified(true);
     };