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

Jeremy Whiting jpwhiting at kde.org
Tue Feb 3 19:24:07 UTC 2015


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

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
-----

  src/khangman.cpp 094c20ec1a0e21b52ede6c4500a5649fa572bcda 
  src/khangman.kcfg d4f402351881d2777a6f5a2dd25129fee571ed80 
  src/khmthemefactory.cpp 3e1a15f923ef31a8dede30456037f0209f2f3c81 
  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 
  themes/desert/CMakeLists.txt c25958125998b841c54e1c5e9576bb051501bebd 
  themes/notes/CMakeLists.txt PRE-CREATION 
  themes/sea/CMakeLists.txt 9cf46fce4f93f1e7b0860fdc0763b5bb1f3317bf 
  themes/standardthemes.xml 3134a9cb19f83a876084f1173a886af3f0b03cd3 
  themes/winter/CMakeLists.txt 9f1fe312d108a66cb0782c2ef09326820fda2d1f 
  src/khangman.h 481fb5bd5f8ea8577f14fd4b7e2a26e14c6fedcc 

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/09f883e5/attachment.html>


More information about the kde-edu mailing list