[Kde-games-devel] Review Request 111049: KNetWalk QML port

[Ashwin Rajeev]

> On June 20, 2013, 10:03 a.m., Viranch Mehta wrote:
> > src/mainwindow.cpp, lines 142-175
> > <http://git.reviewboard.kde.org/r/111049/diff/4/?file=164227#file164227line142>
> >
> >     can you put these inside main.qml directly? they're doing nothing on the c++ part except forwarding keyboard events to QML. you can use http://qt-project.org/doc/qt-4.8/qml-keys.html

Then users wont be able to configure keys. Now it can be done from "configure shortcuts".


- Ashwin


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


On June 18, 2013, 2:36 p.m., Ashwin Rajeev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111049/
> -----------------------------------------------------------
> 
> (Updated June 18, 2013, 2:36 p.m.)
> 
> 
> Review request for KDE Games, Albert Astals Cid and Viranch Mehta.
> 
> 
> Description
> -------
> 
> QML port
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt d017d1a 
>   src/abstractgrid.h ddd9911 
>   src/cell.h 9254935 
>   src/cell.cpp 7555e1d 
>   src/fielditem.h 91e1a33 
>   src/fielditem.cpp a04ad6e 
>   src/mainwindow.h 2c047b6 
>   src/mainwindow.cpp d7d6fbd 
>   src/qml/CanvasItem.qml PRE-CREATION 
>   src/qml/Cell.qml PRE-CREATION 
>   src/qml/logic.js PRE-CREATION 
>   src/qml/main.qml PRE-CREATION 
>   src/scene.h 7753303 
>   src/scene.cpp 38c276f 
>   src/view.h 9de36b8 
>   src/view.cpp 95fe9b4 
> 
> Diff: http://git.reviewboard.kde.org/r/111049/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashwin Rajeev
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20130620/c9c77cf7/attachment-0001.html>