From ab7d01e6e79600b413a2803bfffdb618a5aab81a Mon Sep 17 00:00:00 2001
From: Edward Brown <edward.brown@stfc.ac.uk>
Date: Tue, 15 May 2018 11:34:27 +0100
Subject: [PATCH] Barebones fix to allow tabbing over uneditable cells.
Re #22263
---
qt/widgets/common/src/Batch/JobTreeView.cpp | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/qt/widgets/common/src/Batch/JobTreeView.cpp b/qt/widgets/common/src/Batch/JobTreeView.cpp
index aee5ab297ba..7f1301b6f77 100644
--- a/qt/widgets/common/src/Batch/JobTreeView.cpp
+++ b/qt/widgets/common/src/Batch/JobTreeView.cpp
@@ -443,9 +443,15 @@ JobTreeView::fromFilteredModel(QModelIndex const &filteredModelIndex) const {
QModelIndex JobTreeView::moveCursor(CursorAction cursorAction,
Qt::KeyboardModifiers modifiers) {
if (cursorAction == QAbstractItemView::MoveNext) {
- return applyNavigationResult(navigation().moveCursorNext(currentIndex()));
+ auto result = navigation().moveCursorNext(currentIndex());
+ while (!result.first && result.second.isValid() && !(result.second.flags() & Qt::ItemIsEditable))
+ result = navigation().moveCursorNext(result.second);
+ return applyNavigationResult(result);
} else if (cursorAction == QAbstractItemView::MovePrevious) {
- return navigation().moveCursorPrevious(currentIndex());
+ auto nextIndex = navigation().moveCursorPrevious(currentIndex());
+ while (nextIndex.isValid() && !(nextIndex.flags() & Qt::ItemIsEditable))
+ nextIndex = navigation().moveCursorPrevious(nextIndex);
+ return nextIndex;
} else {
return QTreeView::moveCursor(cursorAction, modifiers);
}
--
GitLab