From 7a380fa34393c2acb461779e960d744cb3d51bfe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Gr=C3=A4=C3=9Flin?= Date: Thu, 16 Jan 2014 13:02:13 +0100 Subject: [PATCH] Reduce timeout of startupfeedback to a sensible default It's 2014 and we don't have to wait half a minute for an application to start. In fact we mostly get false positives due to applications not handling correctly startup notifications for already running instances (e.g. click on link in email). So let's reduce to a default which doesn't look like a broken setup. REVIEW: 115046 --- effects/startupfeedback/startupfeedback.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/effects/startupfeedback/startupfeedback.cpp b/effects/startupfeedback/startupfeedback.cpp index d700138931..4026e6ee7a 100644 --- a/effects/startupfeedback/startupfeedback.cpp +++ b/effects/startupfeedback/startupfeedback.cpp @@ -69,6 +69,7 @@ static const int FRAME_TO_BLINKING_COLOR[] = { static const QColor BLINKING_COLORS[] = { Qt::black, Qt::darkGray, Qt::lightGray, Qt::white, Qt::white }; +static const int s_startupDefaultTimeout = 5; StartupFeedbackEffect::StartupFeedbackEffect() : m_bounceSizesRatio(1.0) @@ -118,7 +119,7 @@ void StartupFeedbackEffect::reconfigure(Effect::ReconfigureFlags flags) const bool busyCursor = c.readEntry("BusyCursor", true); c = conf.group("BusyCursorSettings"); - m_startupInfo->setTimeout(c.readEntry("Timeout", 30)); + m_startupInfo->setTimeout(c.readEntry("Timeout", s_startupDefaultTimeout)); const bool busyBlinking = c.readEntry("Blinking", false); const bool busyBouncing = c.readEntry("Bouncing", true); if (!busyCursor)