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
