--===============0543281961783906323== Content-Type: multipart/alternative; boundary="===============3630244888972696348==" --===============3630244888972696348== 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/103735/#review9963 ----------------------------------------------------------- This review has been submitted with commit a82bb8cb44f1c3e82f38d51f2867452d= 9cf54d9b by Ignat Semenov to branch master. - Commit Hook On Jan. 20, 2012, 9:36 a.m., Ignat Semenov wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103735/ > ----------------------------------------------------------- > = > (Updated Jan. 20, 2012, 9:36 a.m.) > = > = > Review request for Plasma, Aaron J. Seigo and Marco Martin. > = > = > Description > ------- > = > Currently, the FolderView::IconView::itemSize() method accounts for the m= argins only if size.width()>ts.width(), that is, the icon+margins is wider = than the text. However, when ts.width()>size.width(), the margins are not a= ccounted for, and size.width()=3Dts.width(). This can be observed easily. T= he patch adds those margins to the second operand of the comparison macro. > = > = > Diffs > ----- > = > plasma/applets/folderview/iconview.cpp 09b9c80 = > = > Diff: http://git.reviewboard.kde.org/r/103735/diff/diff > = > = > Testing > ------- > = > Tested by manually changing the paintItem() code to use the itemSize() re= ct and observing the improved behaviour. > = > = > Thanks, > = > Ignat Semenov > = > --===============3630244888972696348== 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/103735/ |
This revie= w has been submitted with commit a82bb8cb44f1c3e82f38d51f2867452d9cf54d9b b= y Ignat Semenov to branch master.
- Commit
On January 20th, 2012, 9:36 a.m., Ignat Semenov wrote:
Review request for Plasma, Aaron J. Seigo and Marco Martin.
By Ignat Semenov.
Updated Jan. 20, 2012, 9:36 a.m. Descripti= on
Testing <= /h1>
Diffs=
|