[Kde-games-devel] Review Request: kmahjongg: select the wrong Stone if i click on the shadow of another stone

[Frederik Schwarzer]

> On Dec. 2, 2011, 6:30 a.m., Frederik Schwarzer wrote:
> > I will install this patch later and see if I understand it better then. For now, just this:

>From the user's point of view it is an improvement. However, I can still click around half of the shadow side-ways.
I am not sure I understand the code but do you use the width of the outside edge and cut that from the position? Or do I understand offsetX and offsetY wrong?

Reading the code in general is quite an adventure. I wanted to look at your code but got stuck several times on the way down. ;)
Reformatting the whole thing might be a good idea at some point.
I even found German comments. :D


- Frederik


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6839/#review10497
-----------------------------------------------------------


On Nov. 25, 2011, 7:15 p.m., Christian Krippendorf wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6839/
> -----------------------------------------------------------
> 
> (Updated Nov. 25, 2011, 7:15 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Description
> -------
> 
> Bug List Link: https://bugs.kde.org/show_bug.cgi?id=286927
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdegames/kmahjongg/boardwidget.h 1265765 
>   trunk/KDE/kdegames/kmahjongg/boardwidget.cpp 1265765 
> 
> Diff: http://svn.reviewboard.kde.org/r/6839/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Christian Krippendorf
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20111202/1224a124/attachment.html>