Re: Snapshot of quicklaunch applet

2009-04-10 Thread Aaron J. Seigo
On Friday 10 April 2009, you wrote:
> Hi Aron,
>
> I am sending the snapshot of the quicklaunch applet.
> http://imagebin.org/45059
>
>  In the constructor I had called kDebug() << "Foo Start"
> and just after the resize(sizeHint(Qt::PreferredSize)) i had called another
> kDebug() << "Bar see" in the method refactorUi().
>
> Even if I change m_timer->start(100) to m_timer->start(10) its not making
> much difference :(

currently your patch is only modifying how many times refactorUi is called due 
to changes in size.

but there are many other places in the code that call refactorUi directly.

if you wish to compress _all_ calls to refactorUi, then i would suggest 
introducing a new performUiRefactor() slot that does what refactorUi currently 
does, connect the timer to that slot and make refactorUi start the timer :)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software



signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Snapshot of quicklaunch applet

2009-04-10 Thread sujith h
Hi Aron,

I am sending the snapshot of the quicklaunch applet.
http://imagebin.org/45059

 In the constructor I had called kDebug() << "Foo Start"
and just after the resize(sizeHint(Qt::PreferredSize)) i had called another
kDebug() << "Bar see" in the method refactorUi().

Even if I change m_timer->start(100) to m_timer->start(10) its not making
much difference :(

Sujith H

-- 
സുജിത് ഹരിദാസന്
Bangalore
http://fci.wikia.com/wiki/Anti-DRM-Campaign
 http://sujithh.info
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel