From c9fd8fc1e57dd8b676679e79db58cd64065f0bca Mon Sep 17 00:00:00 2001 From: Kai Uwe Broulik Date: Sun, 8 Mar 2015 19:57:25 +0100 Subject: [PATCH] When changing "active" load pixmap right away The active property is usualy set in response to a hover event. Loading the new pixmap right away, starting the fade animation immediately, provides a more responsive user experience. REVIEW: 122857 --- src/declarativeimports/core/iconitem.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/declarativeimports/core/iconitem.cpp b/src/declarativeimports/core/iconitem.cpp index 03f40c4d9..d653bf382 100644 --- a/src/declarativeimports/core/iconitem.cpp +++ b/src/declarativeimports/core/iconitem.cpp @@ -209,7 +209,7 @@ void IconItem::setActive(bool active) m_active = active; if (isComponentComplete()) { - m_loadPixmapTimer.start(); + loadPixmap(); } emit activeChanged(); }