----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125291/#review85586 -----------------------------------------------------------
do you intend to test for "QWindow::flags() & Qt::FramelessWindowHint"? (This would cover all unmanaged windows as well as docks and the pseudo-popups/tooltips, plasmashell seems to be so addicted to) - Thomas Lübking On Sept. 17, 2015, 4:57 nachm., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125291/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2015, 4:57 nachm.) > > > Review request for kwin and Plasma. > > > Repository: plasma-framework > > > Description > ------- > > the reason popups and tooltips position themselves some pixels lower than > they should is that dialog has a check if the window that shows the item it's > positioning relative to is a panel. but is using kwininfo for that, that of > course doesn't work on wayland. > currently the patch checks with the metaobject if the qwindow inherits > "panelview" and it works, but it's quite horrible. > can't use kwayland from there so don't know a proper way to do it.. maybe > will end up exporting some magic property... > > > Diffs > ----- > > src/plasmaquick/dialog.cpp 8d4f508 > > Diff: https://git.reviewboard.kde.org/r/125291/diff/ > > > Testing > ------- > > > Thanks, > > Marco Martin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
