From kde-core-devel  Thu Sep 17 15:50:39 2009
From: "Adenilson Cavalcanti" <cavalcantii () gmail ! com>
Date: Thu, 17 Sep 2009 15:50:39 +0000
To: kde-core-devel
Subject: Re: Review Request: Adding kinetic scrolling in plasma::ScrollWidget
Message-Id: <20090917155039.20239.14455 () localhost>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=125320269704869


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1614/
-----------------------------------------------------------

(Updated 2009-09-17 15:50:38.966065)


Review request for kdelibs, Aaron Seigo, Marco Martin, and igorto.


Changes
-------

Coding style changes.


Summary
-------

Adding kinetic scrolling in plasma::ScrollWidget using timers.


Diffs (updated)
-----

  svn://anonsvn.kde.org/home/kde/trunk/KDE/kdelibs/plasma/CMakeLists.txt 1024918 
  svn://anonsvn.kde.org/home/kde/trunk/KDE/kdelibs/plasma/private/kineticscroll.cpp PRE-CREATION 
  svn://anonsvn.kde.org/home/kde/trunk/KDE/kdelibs/plasma/private/kineticscroll_p.h PRE-CREATION 
  svn://anonsvn.kde.org/home/kde/trunk/KDE/kdelibs/plasma/widgets/scrollwidget.h 1024918 
  svn://anonsvn.kde.org/home/kde/trunk/KDE/kdelibs/plasma/widgets/scrollwidget.cpp 1024918 

Diff: http://reviewboard.kde.org/r/1614/diff


Testing
-------

Up until now, only tested it with uBlog and plasma-netbook, not sure how it will work with another plasma apps.

It was also tested (I mean the pure Qt version of kinetic scrolling list: http://repo.or.cz/w/kineticlist.git) in a N810 with Qt 4.5.2 and it runs pretty smoothly.


Thanks,

Adenilson