From fa1b5eebb51f4bc372e8ab3bebfa7600b0284641 Mon Sep 17 00:00:00 2001 From: Ulf Hermann <ulf.hermann@digia.com> Date: Wed, 26 Mar 2014 11:41:46 +0100 Subject: [PATCH] QmlProfiler: unify centering on items and do it only once per click Change-Id: Ic228a825aebe3d24eeb5b1d84c58499bae4ae69b Reviewed-by: Kai Koehne <kai.koehne@digia.com> --- src/plugins/qmlprofiler/qml/MainView.qml | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/src/plugins/qmlprofiler/qml/MainView.qml b/src/plugins/qmlprofiler/qml/MainView.qml index 482dd03594d..2d61ba7a737 100644 --- a/src/plugins/qmlprofiler/qml/MainView.qml +++ b/src/plugins/qmlprofiler/qml/MainView.qml @@ -177,7 +177,6 @@ Rectangle { view.selectedItem = itemIndex; if (itemIndex !== -1) { view.selectionLocked = true; - recenterOnItem(modelIndex, itemIndex); } } lockItemSelection = false; @@ -339,19 +338,7 @@ Rectangle { rangeDetails.setLocation(qmlProfilerModelProxy.getEventLocation(selectedModel, selectedItem)); // center view (horizontally) - var windowLength = view.endTime - view.startTime; - var eventStartTime = qmlProfilerModelProxy.getStartTime(selectedModel, selectedItem); - var eventEndTime = eventStartTime + - qmlProfilerModelProxy.getDuration(selectedModel, selectedItem); - - if (eventEndTime < view.startTime || eventStartTime > view.endTime) { - var center = (eventStartTime + eventEndTime)/2; - var from = Math.min(qmlProfilerModelProxy.traceEndTime()-windowLength, - Math.max(0, Math.floor(center - windowLength/2))); - - zoomControl.setRange(from, from + windowLength); - } - + recenterOnItem(selectedModel, selectedItem); if (!lockItemSelection) { lockItemSelection = true; // update in other views -- GitLab