Review Request 121438: change in main.qml

Jeremy Whiting jpwhiting at kde.org
Wed Dec 10 16:29:40 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121438/#review71727
-----------------------------------------------------------

Ship it!


Looks great, but we should keep these changes you are making on your own branch or a clone of kanagram, kanagram master branch needs to stay complete and release-ready. We can discuss on irc where you want to push this to.

- Jeremy Whiting


On Dec. 10, 2014, 8:41 a.m., Souvik  Das wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121438/
> -----------------------------------------------------------
> 
> (Updated Dec. 10, 2014, 8:41 a.m.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Repository: kanagram
> 
> 
> Description
> -------
> 
> Final optimised code for icon toggle in main.qml
> 
> 
> Diffs
> -----
> 
>   src/kanagram.kcfg 468e316528c413b8dc42d2e9d365b6fe63c63257 
>   src/kanagramgame.h e9564cfcc6a19a3ed5e63db5340cd19a760914d5 
>   src/kanagramgame.cpp e0b60866ef49af3f28161c72982b17c6865f88ef 
>   src/ui/icons/1player.png PRE-CREATION 
>   src/ui/icons/2player.png PRE-CREATION 
>   src/ui/main.qml 38408ae66772a578a45faeb92ff9828f7c174f2a 
> 
> Diff: https://git.reviewboard.kde.org/r/121438/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Souvik  Das
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20141210/6a6ce8e1/attachment.html>


More information about the kde-edu mailing list