--===============0293831718016559849== 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/#review8681 ----------------------------------------------------------- Ship it! if this does indeed fix the problem, then let's ship it as this is a seriou= s issue people run into regularly. i wish there was a better fix than just = disabling the horizontal scrollbar, but i'd rather see this not hit users t= han wait for the "perfect" solution that so far has not been forthcoming. - Aaron J. Seigo On Dec. 3, 2011, 10:55 a.m., Jaime Torres Amate wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103313/ > ----------------------------------------------------------- > = > (Updated Dec. 3, 2011, 10:55 a.m.) > = > = > Review request for kdelibs. > = > = > Description > ------- > = > Basically, what I do is: > If there are one or zero columns, hide the horizontalScrollBar until it i= s 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 no= t test with amarok (I've hit by an amarok start bug). > Please, test with other programs. > = > = > Thanks, > = > Jaime Torres Amate > = > --===============0293831718016559849== 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/ |
Ship it!
if this do= es indeed fix the problem, then let's ship it as this is a serious issu= e people run into regularly. i wish there was a better fix than just disabl= ing the horizontal scrollbar, but i'd rather see this not hit users tha= n wait for the "perfect" solution that so far has not been forthc= oming.
- Aaron J.
On December 3rd, 2011, 10:55 a.m., Jaime Torres Amate wrote:
Review request for kdelibs.
By Jaime Torres Amate.
Updated Dec. 3, 2011, 10:55 a.m. Descripti= on
Testing <= /h1>
Diffs=
|