diff options
| author | Dentomologist <dentomologist@gmail.com> | 2024-08-25 15:33:41 -0700 |
|---|---|---|
| committer | Dentomologist <dentomologist@gmail.com> | 2024-08-25 23:44:17 -0700 |
| commit | ee35aa49a2800a341906dbe3cde7626fd0d7849f (patch) | |
| tree | b9df400e32f27160ec06a36d79bdad0060a4f80f /Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp | |
| parent | 5af0ae25e6ebeaec187950f579f722637546740f (diff) | |
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).
Diffstat (limited to 'Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp')
| -rw-r--r-- | Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp b/Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp index d87162214d..b365e8220c 100644 --- a/Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp +++ b/Source/Core/DolphinQt/Config/GeckoCodeWidget.cpp @@ -166,17 +166,16 @@ void GeckoCodeWidget::ConnectWidgets() void GeckoCodeWidget::OnSelectionChanged() { - auto items = m_code_list->selectedItems(); - + const QList<QListWidgetItem*> items = m_code_list->selectedItems(); const bool empty = items.empty(); - m_edit_code->setEnabled(!empty); - m_remove_code->setEnabled(!empty); + m_edit_code->setDisabled(empty); + m_remove_code->setDisabled(empty); - if (items.empty()) + if (empty) return; - auto selected = items[0]; + const QListWidgetItem* const selected = items[0]; const int index = selected->data(Qt::UserRole).toInt(); |
