[Kde-games-devel] Review Request 125498: Do not delete button box too early

Albert Astals Cid aacid at kde.org
Mon Oct 12 00:16:17 UTC 2015


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


This is my variant of the fix

https://paste.kde.org/pag4yewyt

What do you think?

- Albert Astals Cid


On oct. 3, 2015, 2:12 a.m., Frederik Schwarzer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125498/
> -----------------------------------------------------------
> 
> (Updated oct. 3, 2015, 2:12 a.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Repository: libkdegames
> 
> 
> Description
> -------
> 
> As discussed on the mailing list, the buttons just disappear when pressing Enter on the name input in kscoredialog. This was caused by a strange single-shot timer that led to the deletion of the button box.
> I also moved the creation of the Close button to the slot executed when the name input is confirmed, since in the other slots the button was created just before the dialog was closed due to the ::acept and ::reject actions.
> 
> Please comment since I lack experience. :)
> 
> 
> Diffs
> -----
> 
>   highscore/kscoredialog.cpp 5e50948 
> 
> Diff: https://git.reviewboard.kde.org/r/125498/diff/
> 
> 
> Testing
> -------
> 
> Compiled and tested dialog from kshisen by
> - pressing enter -> close button will be shown
> - clicking "forget" -> dialog closes
> - clicking "remember" -> dialog closes
> 
> 
> Thanks,
> 
> Frederik Schwarzer
> 
>

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


More information about the kde-games-devel mailing list