D9805: Show error message to the user
Frederik Schwarzer
noreply at phabricator.kde.org
Wed Jan 10 23:35:26 UTC 2018
schwarzer added a comment.
In https://phabricator.kde.org/D9805#189154, @aacid wrote:
> I just realzied this is not how you use QMEsasgebox critical.
>
> Since it's static you just do QMessageBox::critical(...)
Thanks for pointing that out.
> Second one should be Prefs::background and not Prefs::tileSet?
Of course. Copy&Paste error. -_-
Thanks for the review. I will upload an updated patch tomorrow. :)
REVISION DETAIL
https://phabricator.kde.org/D9805
To: schwarzer, #kde_games
Cc: aacid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20180110/93d7d9c7/attachment.html>
More information about the kde-games-devel
mailing list