[Differential] [Accepted] D2795: Improve KReportExample main window by combining designer and preview widgets

[piggz (Adam Pigg)]

piggz accepted this revision.
piggz added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> window.cpp:66
> +    // restore visual settings
> +    QSettings settings("kde.org", "KReportExample");
> +    restoreGeometry(settings.value("MainWindow/geometry").toByteArray());

Better to use QCoreApplication::setOrganisationName() etc ?
eg http://doc.qt.io/qt-5/qsettings.html#details
Then just call 
QSettings settings;

REPOSITORY
  rKREPORT KReport

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D2795

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: staniek, piggz
Cc: Kexi-Devel-list, staniek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kexi-devel/attachments/20160918/e7bcb72e/attachment.html>