> On Nov. 9, 2011, 6:48 a.m., Fredrik Höglund wrote: > > kcontrol/input/xcursor/cursortheme.cpp, line 119 > > <http://git.reviewboard.kde.org/r/101701/diff/2/?file=33417#file33417line119> > > > > Scaling a pixmap is more expensive than scaling an image. Internally it > > involves converting the pixmap back to an image, scaling it, and then > > converting the image back to a pixmap. > > > > This is not a major issue, but it's the reason the image was scaled > > before it was converted to a pixmap. > > > > Lukas Sommer wrote: > The problem is that I can't drop createIcon(int size) because I need it > on other places. So I would either have to copy the code to createIcon() or > change the return type of createIcon to QImage. Maybe we can leave this for a > later revision? > > Fredrik Höglund wrote: > Like I said, it's not an issue that needs to be fixed now.
A patch is available as review request #104077 but as there, it was commented that this is no longer necessary with Qt 4.8, I have discarted the review request. If this is not okay, let me know. - Lukas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101701/#review8036 ----------------------------------------------------------- On Sept. 2, 2011, 4:40 p.m., Lukas Sommer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101701/ > ----------------------------------------------------------- > > (Updated Sept. 2, 2011, 4:40 p.m.) > > > Review request for KDE Base Apps, KDE Runtime, kdelibs, and Christoph Feck. > > > Description > ------- > > X11 mouse cursor themes can contain cursors in multiple sizes, making them > pseudo-scalable. > > It is yet possible in KDE to configure manually the mouse cursor size > (editing kcminput.rc). However, the GUI of the corresponding KControl module > didn't provide support to change this. This patch add support for changing > the mouse cursor size to the GUI. > > This are mostly GUI related changes. The underlying data structure > XCursorTheme did yet provide support for choosing different sizes and only > needed some adjustments. > > > This addresses bug 90444. > http://bugs.kde.org/show_bug.cgi?id=90444 > > > Diffs > ----- > > kcontrol/input/xcursor/cursortheme.h 586ccba > kcontrol/input/xcursor/cursortheme.cpp 92abea5 > kcontrol/input/xcursor/legacytheme.h 846bf9b > kcontrol/input/xcursor/previewwidget.h f4d2c4e > kcontrol/input/xcursor/previewwidget.cpp 3c264fc > kcontrol/input/xcursor/themepage.h 38ca893 > kcontrol/input/xcursor/themepage.cpp 6c9f29a > kcontrol/input/xcursor/themepage.ui 2e38054 > kcontrol/input/xcursor/xcursortheme.h b474086 > kcontrol/input/xcursor/xcursortheme.cpp 2ecb9ba > > Diff: http://git.reviewboard.kde.org/r/101701/diff/ > > > Testing > ------- > > Tested locally. Works fine for me. Also when using non-standard font DPI > values. > > > Screenshots > ----------- > > > http://git.reviewboard.kde.org/r/101701/s/248/ > > > Thanks, > > Lukas Sommer > >