--===============5448132175614076553== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- 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=3D213068 http://bugs.kde.org/show_bug.cgi?id=3D287847 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 --===============5448132175614076553== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
This is an automatically generated e-mail. To reply, visit: http://git.revie= wboard.kde.org/r/103313/

Review request for kdelibs.
By Jaime Torres Amate.

Descripti= on 

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.

Testing <= /h1> 
Krunner config does not loop (neither kgetnewstuff from ksta=
rs). I can not test with amarok (I've hit by an amarok start bug).
Please, test with other programs.
Bugs: 213068, = 287847

Diffs=

  • kdeui/itemviews/kcategorizedview.cpp (46a1= cde)
  • kutils/kpluginselector.cpp (ca0691d)

View Diff
--===============5448132175614076553==--