> On July 17, 2014, 1:42 p.m., Martin Gräßlin wrote: > > would it be possible to add a unit test for it? I know it's not easy as it > > depends on the window manager, but Openbox which is used on the CI system > > is quite decent. > > Martin Klapetek wrote: > I may try
Please add/close bug 337353. - Christoph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119337/#review62576 ----------------------------------------------------------- On July 17, 2014, 1:21 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119337/ > ----------------------------------------------------------- > > (Updated July 17, 2014, 1:21 p.m.) > > > 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