yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Reduce timeout of startupfeedback to a sensible default

Browse source 
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
This commit is contained in:
Martin Gräßlin 2014-01-16 13:02:13 +01:00
parent 551c0c49e6
commit 7a380fa343
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
effects/startupfeedback/startupfeedback.cpp
View file

@ -69,6 +69,7 @@ static const int FRAME_TO_BLINKING_COLOR[] = {
static const QColor BLINKING_COLORS[] = { static const QColor BLINKING_COLORS[] = {
Qt::black, Qt::darkGray, Qt::lightGray, Qt::white, Qt::white Qt::black, Qt::darkGray, Qt::lightGray, Qt::white, Qt::white
}; };
static const int s_startupDefaultTimeout = 5;
StartupFeedbackEffect::StartupFeedbackEffect() StartupFeedbackEffect::StartupFeedbackEffect()
: m_bounceSizesRatio(1.0) : m_bounceSizesRatio(1.0)
@ -118,7 +119,7 @@ void StartupFeedbackEffect::reconfigure(Effect::ReconfigureFlags flags)
const bool busyCursor = c.readEntry("BusyCursor", true); const bool busyCursor = c.readEntry("BusyCursor", true);
c = conf.group("BusyCursorSettings"); 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 busyBlinking = c.readEntry("Blinking", false);
const bool busyBouncing = c.readEntry("Bouncing", true); const bool busyBouncing = c.readEntry("Bouncing", true);
if (!busyCursor) if (!busyCursor)