----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119337/ -----------------------------------------------------------
(Updated Jan. 20, 2015, 4:46 p.m.) Status ------ This change has been discarded. Review request for KDE Frameworks. Repository: kxmlgui Description ------- Playing a bit with the KXmlGuiWindow test showed that it didn't restore its size properly on reopening. I found out that KWindowConfig::saveWindowSize() is called before KWindowConfig::restoreWindowSize() which results in overwriting the values in the config file and KWindowConfig::restoreWindowSize() then reads the changed values. So this patch calls KWindowConfig::saveWindowSize() only after we have applied the size from the config file. Additionally, it checks if the windowHandle() we're trying to resize is NULL as that can result in no resize and yet the d->sizeApplied being wrongly set to true. Diffs ----- src/kmainwindow.cpp e273a76 Diff: https://git.reviewboard.kde.org/r/119337/diff/ Testing ------- KXmlGuiWindow (the one in tests/) now correctly stores & restores its size. Thanks, Martin Klapetek
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel