From 9772590e9adb361a2be7aade109c3f6f6ecfd5dc Mon Sep 17 00:00:00 2001 From: Hugh Davenport Date: Mon, 13 May 2024 14:16:09 +1200 Subject: [PATCH] MouseSettings: Only allow valid cursor themes To align the cursor theme tab to how DisplaySettings themes tab works, this change forces the theme combo box to not allow free-text. Currently on a click it puts the text cursor in the box to allow typing anything rather than acting as a dropdown when clicking anywhere on the field. Fixes #24306 --- Userland/Applications/MouseSettings/ThemeWidget.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/Userland/Applications/MouseSettings/ThemeWidget.cpp b/Userland/Applications/MouseSettings/ThemeWidget.cpp index 47a2ce08ab..4d3fa458de 100644 --- a/Userland/Applications/MouseSettings/ThemeWidget.cpp +++ b/Userland/Applications/MouseSettings/ThemeWidget.cpp @@ -135,6 +135,7 @@ ErrorOr ThemeWidget::initialize() m_mouse_cursor_model->change_theme(theme_name); m_theme_name_box = find_descendant_of_type_named("theme_name_box"); + m_theme_name_box->set_only_allow_values_from_model(true); m_theme_name_box->on_change = [this](ByteString const& value, GUI::ModelIndex const&) { m_mouse_cursor_model->change_theme(value); set_modified(true);