Review Request: fix infinite recurssion in kcategorizedview
Jaime Torres Amate
jtamate at gmail.com
Sat Dec 3 10:55:43 GMT 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103313/
-----------------------------------------------------------
Review request for kdelibs.
Description
-------
Basically, what I do is:
If there are one or zero columns, hide the horizontalScrollBar until it is needed. (it has worked in the past, but in another file).
Apply the same strategy with files.
Additional stuff:
Moved the common calculus of itemSize outside of the if then else.
This addresses bugs 213068 and 287847.
http://bugs.kde.org/show_bug.cgi?id=213068
http://bugs.kde.org/show_bug.cgi?id=287847
Diffs
-----
kdeui/itemviews/kcategorizedview.cpp 46a1cde
kutils/kpluginselector.cpp ca0691d
Diff: http://git.reviewboard.kde.org/r/103313/diff/diff
Testing
-------
Krunner config does not loop (neither kgetnewstuff from kstars). I can not test with amarok (I've hit by an amarok start bug).
Please, test with other programs.
Thanks,
Jaime Torres Amate
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20111203/e7da67fb/attachment.htm>
More information about the kde-core-devel
mailing list