D13808: Fix KMainWindow saving incorrect widget settings

[Wolfgang Bauer]

wbauer added a comment.


  In D13808#286965 <https://phabricator.kde.org/D13808#286965>, @maxrd2 wrote:
  
  > Relative Qt Bug is here: https://bugreports.qt.io/browse/QTBUG-69277
  >  Seems it's not their bug.
  
  
  Hm.
  Interestingly, Qt Creator has similar problems with Qt 5.11.1 that are also fixed by reverting that commit in Qt:
  http://bugzilla.opensuse.org/show_bug.cgi?id=1099752
  (and Qt Creator also doesn't use kxmlgui AFAIK... ;-) )
  
  And the second last comment in https://codereview.qt-project.org/#/c/227398/ says:
  
  > This change somehow breaks QMainWindow::saveState() when it's called from the destructor: toolbars are not shown on the next start.
  
  Anyway, that's not really related to this review, or whether the proposed change makes sense...

REPOSITORY
  R263 KXmlGui

BRANCH
  fix-window-state-save

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

To: maxrd2, #kde_applications, dfaure, elvisangelaccio, broulik, cfeck
Cc: wbauer, aacid, ngraham, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180705/7a0c39d5/attachment-0001.html>