Review Request 122412: Add themes back into the new qml ui.

Rahul Chowdhury rahul.chowdhury at kdemail.net
Tue Feb 3 19:34:24 UTC 2015


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

Ship it!


Ship It!

- Rahul Chowdhury


On Feb. 3, 2015, 7:32 p.m., Jeremy Whiting wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122412/
> -----------------------------------------------------------
> 
> (Updated Feb. 3, 2015, 7:32 p.m.)
> 
> 
> Review request for KDE Edu, Laszlo Papp and Rahul Chowdhury.
> 
> 
> Repository: khangman
> 
> 
> Description
> -------
> 
> Default to "notes" theme from the harmattan ui.
> Add theme choosing in the bottom toolbar.
> Remove unused categoriesList method.
> Install themes into khangman/themes/ rather than khangman/pics.
> Added notes theme to standardthemes.xml file.
> Changed background image from TileVertically to default Stretch.
> Added new properties in khangman class as needed for currenttheme, themes
>  and backgroundUrl.
> 
> TODO: Use theme letter colors and possibly placement also.
> 
> 
> Diffs
> -----
> 
>   themes/sea/CMakeLists.txt 9cf46fce4f93f1e7b0860fdc0763b5bb1f3317bf 
>   themes/standardthemes.xml 3134a9cb19f83a876084f1173a886af3f0b03cd3 
>   themes/winter/CMakeLists.txt 9f1fe312d108a66cb0782c2ef09326820fda2d1f 
>   themes/desert/CMakeLists.txt c25958125998b841c54e1c5e9576bb051501bebd 
>   themes/notes/CMakeLists.txt PRE-CREATION 
>   src/qml/GamePage.qml 7d60baad978d93198a0778bae48ee64c1240ac9d 
>   src/qml/khangman-background-landscape.png  
>   src/qml/khangman-background-portrait.png  
>   src/qml/main.qml 05163836caa636f383a964d7dc7dfb4324447938 
>   themes/CMakeLists.txt f2a58a5758e3fa6cb3c70f56e9075ad57732a674 
>   themes/bees/CMakeLists.txt c127574d41b105c64513c13b926b9b3ed25b01ae 
>   src/khangman.h 481fb5bd5f8ea8577f14fd4b7e2a26e14c6fedcc 
>   src/khangman.cpp 094c20ec1a0e21b52ede6c4500a5649fa572bcda 
>   src/khangman.kcfg d4f402351881d2777a6f5a2dd25129fee571ed80 
>   src/khmthemefactory.cpp 3e1a15f923ef31a8dede30456037f0209f2f3c81 
> 
> Diff: https://git.reviewboard.kde.org/r/122412/diff/
> 
> 
> Testing
> -------
> 
> It builds, runs and theme changing works as far as I've checked. I've tried every theme and tried with no ~/.config/khangmanrc in place.
> I would like to add the author of the notes theme, I think it was Eugene, but can't tell from the git log.
> 
> 
> Thanks,
> 
> Jeremy Whiting
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20150203/2a7e4a26/attachment.html>


More information about the kde-edu mailing list