[Kde-games-devel] KBreakout's QML port

[Viranch Mehta]

Hey,

On 29-Dec-2012, at 12:58 AM, Albert Astals Cid wrote:

> Hi Viranch, i just found some time to give it a try and it works pretty well, 
> congrats :-)
> 
> Three things that i found:
> * Mouse can be moved outside the window when playing with the mouse, that's 
> pretty bad and a regression from the old behaviour, do you think you can have 
> a look at this?

capturing and controlling mouse movements is a little difficult with QML. I can do these with the C++ part but then the mouse events won't reach the QML view (not directly anyway, will have to write hacks for forwarding the events which can get messy).

Also, I found that mouse controlling thing in the old version very annoying at initial uses. I just wanted to close the game or go to some menu and it just won't let me, took me some time to figure out i have to first pause the game to get back the control.

> * Randomly the score text seems to get bigger (like double size or something) 
> for like 0.3 seconds, it's hard to see since you are usually looking at the 
> ball and not the score but i got some flashiness on the border of my visual 
> camp and after looking at it i found it was that. I see it flash like 4 or 5 
> times while playing 3 levels.

yes I have noticed this. This is because it has to calculate the maximum size of the score text which can be fit in the box. so how this happens is every time score text is changed, it makes its font size to 72 (very big) and then reduces it until the text fits the box. this is how the C++ version also implemented it, but it was faster then.

> * You seem to have a broken connect, when opening the configure dialog i get
> Object::connect: No such slot MainWindow::loadSettings()
> Object::connect:  (sender name:   'settings')
> Object::connect:  (receiver name: 'MainWindow#1')
> Wops, now i see that was already there in the old version, anyway could you 
> fix it (may as well just be removing the connect if we don't really need it)

will have a look at it.

> Good work overall :-)

Thanks! :)

Cheers,
Viranch