----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118199/#review58146 -----------------------------------------------------------
So between Qt4->5 there must have been ensurePolished() removed from QWidget. We restored it in QMenu, but perhaps that wasn't actually the right approach? - David Edmundson On May 19, 2014, 9:49 a.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118199/ > ----------------------------------------------------------- > > (Updated May 19, 2014, 9:49 a.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > [klipper] Call ensurePolished() before native window of menu gets created > > It's important to have ensurePolished() being called before the native > XCB window gets created. The widget style (in our case Oxygen) sets the > RGBA hint during that call. Once the native window is created this would > fail. > > In the case of KlipperPopup the native window gets created by accessing > the winId(), thus the ensurePolished() needs to happen before. > > > Diffs > ----- > > klipper/klipperpopup.cpp 574f2d7a0d0b30f94c72b2e2715778f619539a00 > > Diff: https://git.reviewboard.kde.org/r/118199/diff/ > > > Testing > ------- > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
