From ee35aa49a2800a341906dbe3cde7626fd0d7849f Mon Sep 17 00:00:00 2001 From: Dentomologist Date: Sun, 25 Aug 2024 15:33:41 -0700 Subject: ARWidget: Disable Edit and Remove buttons when no code is selected Also some minor refactoring of nearby/related code: * Make non-obvious variable types explicit instead of auto. * Throw some consts around. * Use setDisabled(empty) instead of setEnabled(!empty). --- Source/Core/DolphinQt/Config/ARCodeWidget.cpp | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) (limited to 'Source/Core/DolphinQt/Config/ARCodeWidget.cpp') diff --git a/Source/Core/DolphinQt/Config/ARCodeWidget.cpp b/Source/Core/DolphinQt/Config/ARCodeWidget.cpp index d40d4f9109..f42a33317b 100644 --- a/Source/Core/DolphinQt/Config/ARCodeWidget.cpp +++ b/Source/Core/DolphinQt/Config/ARCodeWidget.cpp @@ -67,8 +67,8 @@ void ARCodeWidget::CreateWidgets() m_code_list->setEnabled(!m_game_id.empty()); m_code_add->setEnabled(!m_game_id.empty()); - m_code_edit->setEnabled(!m_game_id.empty()); - m_code_remove->setEnabled(!m_game_id.empty()); + m_code_edit->setEnabled(false); + m_code_remove->setEnabled(false); m_code_list->setContextMenuPolicy(Qt::CustomContextMenu); @@ -179,14 +179,18 @@ void ARCodeWidget::OnListReordered() void ARCodeWidget::OnSelectionChanged() { - auto items = m_code_list->selectedItems(); + const QList items = m_code_list->selectedItems(); + const bool empty = items.empty(); - if (items.empty()) + m_code_edit->setDisabled(empty); + m_code_remove->setDisabled(empty); + + if (empty) return; - const auto* selected = items[0]; + const QListWidgetItem* const selected = items[0]; - bool user_defined = m_ar_codes[m_code_list->row(selected)].user_defined; + const bool user_defined = m_ar_codes[m_code_list->row(selected)].user_defined; m_code_remove->setEnabled(user_defined); m_code_edit->setText(user_defined ? tr("&Edit Code...") : tr("Clone and &Edit Code...")); -- cgit v1.2.3