From 0f67fe1c6b1aa223b5908f1419adbf17a4e39eb8 Mon Sep 17 00:00:00 2001 From: RickyRister Date: Mon, 29 Dec 2025 21:24:24 -0800 Subject: [PATCH] remove debug --- .../deck_editor/deck_editor_deck_dock_widget.cpp | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/cockatrice/src/interface/widgets/deck_editor/deck_editor_deck_dock_widget.cpp b/cockatrice/src/interface/widgets/deck_editor/deck_editor_deck_dock_widget.cpp index 54f3b516e..8301bd626 100644 --- a/cockatrice/src/interface/widgets/deck_editor/deck_editor_deck_dock_widget.cpp +++ b/cockatrice/src/interface/widgets/deck_editor/deck_editor_deck_dock_widget.cpp @@ -572,16 +572,14 @@ void DeckEditorDeckDockWidget::changeSelectedCard(int changeBy) auto deckViewCurrentIndex = deckView->currentIndex(); // For some reason, if the deckModel is modified but the view is not manually reselected, - // currentIndex will return an index for the underlying deckModel instead of the proxy, - // which won't behave as expected when indexBelow/indexAbove crosses a header node. - if (deckViewCurrentIndex.model() == deckModel) { + // currentIndex will return an index for the underlying deckModel instead of the proxy. + // That index will return an invalid index when indexBelow/indexAbove crosses a header node, + // causing the selection to fail to move down. + if (deckViewCurrentIndex.model() == proxy->sourceModel()) { deckViewCurrentIndex = proxy->mapFromSource(deckViewCurrentIndex); } auto nextIndex = deckViewCurrentIndex.siblingAtRow(deckViewCurrentIndex.row() + changeBy); - - qInfo() << "TRACK current:" << deckViewCurrentIndex << "next:" << nextIndex; - if (!nextIndex.isValid()) { nextIndex = deckViewCurrentIndex; @@ -593,9 +591,6 @@ void DeckEditorDeckDockWidget::changeSelectedCard(int changeBy) } else { nextIndex = deckView->indexAbove(nextIndex); } - - qInfo() << "TRACK nextIndex:" << nextIndex; - node = static_cast(nextIndex.internalPointer()); } while (node && node->isDeckHeader()); }