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

[Frederik Schwarzer]

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

(Updated Oct. 12, 2015, 8:40 p.m.)


Status
------

This change has been discarded.


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/063227ea/attachment.html>