Bug 155912 - fifteen puzzle splits up custom images incorrectly
Summary: fifteen puzzle splits up custom images incorrectly
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-fifteenpuzzle (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-16 14:49 UTC by Anthony Bryant
Modified: 2008-01-20 12:24 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
screenshot of bug in fifteen puzzle (99.47 KB, image/png)
2008-01-16 14:53 UTC, Anthony Bryant
Details
patch against revision 763629 (331 bytes, patch)
2008-01-20 01:12 UTC, Anthony Bryant
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Anthony Bryant 2008-01-16 14:49:24 UTC
Version:            (using KDE 4.0.0)
Installed from:    Ubuntu Packages
OS:                Linux

When the fifteen puzzle is set to use a custom image, using "split image into puzzle pieces" in the config, the top right corner of the image is used for the blank tile, instead of the bottom right tile where it should be.

This can be reproduced by solving the puzzle and then setting it to use a custom image. As it is at the moment, it is impossible to solve the puzzle so that the image looks right.
Comment 1 Anthony Bryant 2008-01-16 14:53:18 UTC
Created attachment 23075 [details]
screenshot of bug in fifteen puzzle

Here is a screenshot showing a solved puzzle with a custom image.
Comment 2 Anthony Bryant 2008-01-20 01:12:47 UTC
Created attachment 23148 [details]
patch against revision 763629

The loop was changing to the next row of the image too early, so only a bit of
logic needed changing.
Comment 3 Jesper Thomschütz 2008-01-20 01:15:25 UTC
Seems good to me, feel free to apply the patch.
Comment 4 Anthony Bryant 2008-01-20 01:27:54 UTC
I would if I had an svn account... can someone who does apply it please?
Comment 5 Jesper Thomschütz 2008-01-20 11:26:55 UTC
Whoops, sorry! Revision:763726 is your patch.

Thanks Anthony :)
Comment 6 Anne-Marie Mahfouf 2008-01-20 12:24:18 UTC
Closing as Jesper applied Anthony's patch 
http://lists.kde.org/?l=kde-commits&m=120082460610962&w=2
and it's working properly now.