Review Request 113185: Cursor Theme KCM: Show correct resize cursor in preview for themes without a file called "size_fdiag"

[Wolfgang Bauer]

> On Oct. 10, 2013, 1:21 p.m., Thomas Lübking wrote:
> > https://bugs.kde.org/show_bug.cgi?id=54359
> > If Fredrik's right, the tutorial used by most cursor creators would be wrong (great...)
> > 
> > I guess to be absolutely sure, we'll have to XCreateFontCursor and then somehow get the bitmaps hash (XGetImage and on the bits? No idea...)
> > For the moment i'll trust Fredrik more ;-)

Well, according to https://bugs.kde.org/show_bug.cgi?id=248599#c6, size_fdiag in QCursor does have the hash c7088f0f3e6c8088236ef8e1e3e70000, so my patch would be correct.
I haven't verified that, though.


- Wolfgang


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113185/#review41484
-----------------------------------------------------------


On Oct. 10, 2013, 11:51 a.m., Wolfgang Bauer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113185/
> -----------------------------------------------------------
> 
> (Updated Oct. 10, 2013, 11:51 a.m.)
> 
> 
> Review request for kde-workspace, kwin, Fredrik Höglund, and Thomas Lübking.
> 
> 
> Bugs: 325837
>     http://bugs.kde.org/show_bug.cgi?id=325837
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> Apparently in XCursorTheme::findAlternative() (file kcontrol/input/xcursor/xcursortheme.cpp) the alternatives for "size_bdiag" and "size_fdiag" are swapped, so for themes not containing "size_fdiag" the wrong resize cursor is shown in the preview.
> 
> This patch fixes that long standing bug. (there has been no change to that function since 2007!)
> 
> This also fixes the glitch mentioned in bug#325763, that the wrong arrows are used for the window resize hint after the theme change is applied (for the current X session).
> 
> 
> Diffs
> -----
> 
>   kcontrol/input/xcursor/xcursortheme.cpp 010c9ad 
> 
> Diff: http://git.reviewboard.kde.org/r/113185/diff/
> 
> 
> Testing
> -------
> 
> - Enter systemsettings->Workspace Appearance->Cursor Theme
> - Select a theme without "size_fdiag", f.e.: crystalwhite, DMZ, Adwaita
> - Look at the preview: without the patch, the wrong resize cursor is shown, with the patch it's the same as for Oxygen e.g.
> See atached screenshots
> 
> 
> File Attachments
> ----------------
> 
> KCM without the patch
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/10/10/9cb9ae8c-6614-49ea-aae2-fdbeb36dd71e__cursor.png
> KCM with the patch
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/10/10/f3cf8c6d-d2a0-4e96-8f77-75a53f66395f__cursor2.png
> 
> 
> Thanks,
> 
> Wolfgang Bauer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20131010/88881b7a/attachment.htm>