Review Request: Achievements 20: Add a threshold value to the Achievement class

[Felix Rohrbach]

> On Nov. 26, 2011, 5:46 p.m., Arjen Hiemstra wrote:
> > engine/achievementsmanager.cpp, line 117
> > <http://git.reviewboard.kde.org/r/103253/diff/1/?file=42186#file42186line117>
> >
> >     Would it be an idea to keep the logic to show hidden achievements and instead have the UI decide how to show them? If you completely hide them from the list the UI will never be able to touch them.

Yeah, that's right. It's a bit more work for the delegate, but as all the hidden achievements are at the end, that shouldn't be a big problem.


- Felix


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


On Nov. 26, 2011, 3:47 p.m., Felix Rohrbach wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103253/
> -----------------------------------------------------------
> 
> (Updated Nov. 26, 2011, 3:47 p.m.)
> 
> 
> Review request for Gluon.
> 
> 
> Description
> -------
> 
> With this threshold value you define when the player starts to make progress. Hidden achievements will be shown as soon as the player starts to make progress.
> 
> Further changes:
> - AchievementsManager::dependency now returns the index of the dependency
> - Don't even show hidden elements to the delegate, so that rowCount == achievementsCount.
> 
> This is the last commit in my achievements branch :). After that, only the new delegate (which I have to rewrite) is missing.
> 
> 
> Diffs
> -----
> 
>   engine/achievement.h 5d8403a 
>   engine/achievement.cpp 4a5e544 
>   engine/achievementsmanager.h 785226a 
>   engine/achievementsmanager.cpp ae95ddd 
> 
> Diff: http://git.reviewboard.kde.org/r/103253/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Felix Rohrbach
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/gluon/attachments/20111126/a0d7e88f/attachment.html>