RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-27 12:47:35 +00:00
Code Issues Activity Actions

MailSettings: Make TLS option checked by default

Browse source 
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.
This commit is contained in:
Karol Kosek 2023-07-11 22:41:47 +02:00 • committed by Tim Flynn
parent fe3612ebcb
commit d8bdf26917
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Userland/Applications/MailSettings/MailSettingsWidget.cpp
View file

@ -19,7 +19,7 @@ void MailSettingsWidget::reset_default_values()
{ {
m_server_inputbox->set_text(""sv); m_server_inputbox->set_text(""sv);
m_port_combobox->set_text("993"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); 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 = *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) { m_tls_checkbox->on_checked = [&](auto) {
set_modified(true); set_modified(true);
}; };