--===============1758699202988582640== Content-Type: multipart/alternative; boundary="===============0159434673941739615==" --===============0159434673941739615== 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/106166/#review18186 ----------------------------------------------------------- This review has been submitted with commit 16c54cb9fc4d2085db35c8115aa890f8= aad571b8 by Kai Uwe Broulik to branch master. - Commit Hook On Aug. 24, 2012, 6:17 p.m., Kai Uwe Broulik wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106166/ > ----------------------------------------------------------- > = > (Updated Aug. 24, 2012, 6:17 p.m.) > = > = > Review request for kwin. > = > = > Description > ------- > = > This patch makes KWin=E2=80=99s startup feedback effect aka bouncing curs= or respect the global small icon size. The icon was loaded correctly but sc= aled to a fixed size previously. > (The variable could be given a better name for sure) > = > = > This addresses bug 305611. > http://bugs.kde.org/show_bug.cgi?id=3D305611 > = > = > Diffs > ----- > = > kwin/effects/startupfeedback/startupfeedback.h 6758354 = > kwin/effects/startupfeedback/startupfeedback.cpp ff6a51e = > = > Diff: http://git.reviewboard.kde.org/r/106166/diff/ > = > = > Testing > ------- > = > Tested with 16, 22 and 48 icon size and works fine. > = > = > Thanks, > = > Kai Uwe Broulik > = > --===============0159434673941739615== 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/106166/

This revie=
w has been submitted with commit 16c54cb9fc4d2085db35c8115aa890f8aad571b8 b=
y Kai Uwe Broulik to branch master.

- Commit


On August 24th, 2012, 6:17 p.m., Kai Uwe Broulik wrote:

Review request for kwin.
By Kai Uwe Broulik.

Updated Aug. 24, 2012, 6:17 p.m.

Descripti= on

This patch makes KWin=E2=80=99s startup feedback effect aka =
bouncing cursor respect the global small icon size. The icon was loaded cor=
rectly but scaled to a fixed size previously.
(The variable could be given a better name for sure)

Testing <= /h1>
Tested with 16, 22 and 48 icon size and works fine.
Bugs: 305611

Diffs=

  • kwin/effects/startupfeedback/startupfeedback.h (6758354)
  • kwin/effects/startupfeedback/startupfeedback.cpp (ff6a51e)

View Diff

--===============0159434673941739615==-- --===============1758699202988582640== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ kwin mailing list kwin@kde.org https://mail.kde.org/mailman/listinfo/kwin --===============1758699202988582640==--