From 05fa0cc0dba94b30f8f1c7f4c6479af49d2f96b3 Mon Sep 17 00:00:00 2001 From: Christian Stenger Date: Mon, 24 Oct 2016 09:17:27 +0200 Subject: AutoTest: Let results pane only pop up on finish It is quite annoying if one minimizes results pane while a long test is running and for each result item the pane comes up again. Change-Id: I7020ae698cd61df01ed5b1402cc06d26fb018978 Reviewed-by: David Schulz --- src/plugins/autotest/testresultspane.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/plugins/autotest/testresultspane.cpp b/src/plugins/autotest/testresultspane.cpp index 0f4b6c9c24..d2a8781bcf 100644 --- a/src/plugins/autotest/testresultspane.cpp +++ b/src/plugins/autotest/testresultspane.cpp @@ -208,8 +208,6 @@ void TestResultsPane::addTestResult(const TestResultPtr &result) m_atEnd = scrollBar ? scrollBar->value() == scrollBar->maximum() : true; m_model->addTestResult(result, m_expandCollapse->isChecked()); - if (!m_treeView->isVisible()) - popup(Core::IOutputPane::NoModeSwitch); setIconBadgeNumber(m_model->resultTypeCount(Result::Fail) + m_model->resultTypeCount(Result::UnexpectedPass)); flash(); @@ -498,6 +496,8 @@ void TestResultsPane::onTestRunFinished() m_model->removeCurrentTestMessage(); disconnect(m_treeView->verticalScrollBar(), &QScrollBar::rangeChanged, this, &TestResultsPane::onScrollBarRangeChanged); + if (!m_treeView->isVisible()) + popup(Core::IOutputPane::NoModeSwitch); } void TestResultsPane::onScrollBarRangeChanged(int, int max) -- cgit v1.2.1