From fa14c8dfab7c34ab7a9c94af9e64b620cdaa2e23 Mon Sep 17 00:00:00 2001
From: Henning Gruendl <henning.gruendl@qt.io>
Date: Fri, 16 Feb 2024 18:41:13 +0100
Subject: [PATCH] QmlDesigner: Fix scrollTo multi select behavior

Instead of scrolling to the first element in the selection, scroll to
the last. This will keep the existing behavior for single selection
and will fix the behavior for multi selection.

Task-number: QDS-9794
Change-Id: I846c6ea6b967ef76a77d4ea0fc819d06772eca01
Reviewed-by: Qt CI Patch Build Bot <ci_patchbuild_bot@qt.io>
Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
---
 src/plugins/qmldesigner/components/navigator/navigatorview.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/plugins/qmldesigner/components/navigator/navigatorview.cpp b/src/plugins/qmldesigner/components/navigator/navigatorview.cpp
index 5cea0aab4ea..a3ad444e243 100644
--- a/src/plugins/qmldesigner/components/navigator/navigatorview.cpp
+++ b/src/plugins/qmldesigner/components/navigator/navigatorview.cpp
@@ -700,7 +700,7 @@ void NavigatorView::updateItemSelection()
     blockSelectionChangedSignal(blocked);
 
     if (!selectedModelNodes().isEmpty())
-        treeWidget()->scrollTo(indexForModelNode(selectedModelNodes().constFirst()));
+        treeWidget()->scrollTo(indexForModelNode(selectedModelNodes().constLast()));
 
     // make sure selected nodes are visible
     for (const QModelIndex &selectedIndex : itemSelection.indexes()) {