[Marble-devel] Review Request 123343: MessageBox asking to save tour when closing

[SIlvia Pripoae]

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

(Updated April 27, 2015, 8:56 p.m.)


Review request for Marble, Calin Cruceru and Torsten Rahn.


Changes
-------

The condition now uses the boolean m_isChanged, instead of the action state.


Repository: marble


Description
-------

Before this patch, when closing Marble, any unsaved tours were discarded. Now, a message box asking whether to save the tour is displayed.


Diffs (updated)
-----

  src/apps/marble-kde/KdeMainWindow.h 7a94119 
  src/apps/marble-kde/KdeMainWindow.cpp bebb8a1 
  src/apps/marble-qt/QtMainWindow.h 595ffd2 
  src/apps/marble-qt/QtMainWindow.cpp c4280c6 
  src/apps/marble-ui/ControlView.h 21d58cf 
  src/apps/marble-ui/ControlView.cpp a16aa28 
  src/lib/marble/TourWidget.h a6cb8ca 
  src/lib/marble/TourWidget.cpp b9fab0e 

Diff: https://git.reviewboard.kde.org/r/123343/diff/


Testing
-------

Works for me.


Thanks,

SIlvia Pripoae

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20150427/73bd1fcc/attachment.html>