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

Frederik Schwarzer schwarzer at kde.org
Sat Oct 3 02:12:46 UTC 2015


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

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/20151003/5bf172bc/attachment.html>


More information about the kde-games-devel mailing list