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