Review Request 129238: fix theme selector theme lookup

Harald Sitter sitter at kde.org
Fri Oct 21 14:17:01 UTC 2016


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

Review request for KDE Games.


Repository: libkdegames


Description
-------

knewstuff puts downloaded assets into subdirs like
`.local/share/kblocks/themes/brlublur-hotstuff-access/` which we found
correctly using a subdir iterator, but then failed to preserve the subdir
portion for later loading through KGameTheme in the same function.
to make sure KGameTheme can properly resolve the full path we need to pass
the desktop file path *relative* to data dir we found it in.

CHANGELOG: fix theme selector theme lookup


Diffs
-----

  libkdegamesprivate/kgamethemeselector.cpp 51a9f81d9886f38c56845e3730c50b56accdf95c 

Diff: https://git.reviewboard.kde.org/r/129238/diff/


Testing
-------

theme downloads correctly, then listed in selector, can be applied as theme, gets loaded, still loaded on app restart


Thanks,

Harald Sitter

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20161021/785b5c14/attachment.html>


More information about the kde-games-devel mailing list