[Kde-games-devel] Review Request 113782: Make "current standings" dialogue a dockable widget.

[Nemanja Hirsl]

> On Nov. 11, 2013, 12:48 p.m., Nemanja Hirsl wrote:
> > At the end of the game there is "Final standings" modal dialog which contains the same info as "Standings" dialog. Is there a way to use just one and to end game in different way?
> 
> Alexander Schuch wrote:
>     I do not fully understand what you mean.
>     
>     From a code point of view, there only is one widget for the standings overview. It is used in the dock and in the dialogue.
>     
>     Winning the game (or losing it) is a singular event as the actual game ceases to exist. This deserves modal information. Maybe the standings dock could be closed. But this might be confusing if just the standings dock closes but the messages dock stays open. Closing the messages dock as well might hide information the player is interested in. So in short, I do not fully understand what you mean and about what you could mean, I have no better idea right now.

It was probably because standings overview was by default on and when game ended it appeared again as modal. It looks little odd to have the same table one on top of the another showing the same data. I just wanted to ask if you have any idea how not to use it if it already present?
One way (I'm not saying it is right way) is to put focus on current standings overview if it is visible or to show it, if it is not. This raises new questions do we need modal dialog at the end and what should be there?

I'm OK with current implementation because it is behaving in the same way as it used to, the purpose of this comment is just to ask if there are other (better) ways to handle this situations.  


- Nemanja


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


On Nov. 16, 2013, 5:58 a.m., Alexander Schuch wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113782/
> -----------------------------------------------------------
> 
> (Updated Nov. 16, 2013, 5:58 a.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Repository: konquest
> 
> 
> Description
> -------
> 
> The dialogue "current standings" is replaced by a dockable widget. This way the player is able to see the standings all the time without the need to open/close the dialogue at the beginning of each turn.
> 
> The dockable widget can also be made floating, so it can be used almost the same as the modal dialogue before.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 3b321ad 
>   dialogs/scoredlg.h bda18a5 
>   dialogs/scoredlg.cc 5e66cd4 
>   gameview.h 7d8700e 
>   gameview.cc f67d570 
>   konquestui.rc 683cd11 
>   mainwin.h 663177b 
>   mainwin.cc 2b4cece 
>   view/standingswidget.h PRE-CREATION 
>   view/standingswidget.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/113782/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alexander Schuch
> 
>

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