----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116722/#review52650 -----------------------------------------------------------
Oddly is exactly the patch that i was trying yesterday ;) one thing i found out tough: item->window() of course is not aways there (for instance whenthe item is in a Dialog that hasn't been shown yet) so in many cases this won't work. I wonder if it may be made a bit more complex keeping the item and connecting to windowChanged() to update the window in case is a qquickItem? (and in any way it becomes, the corresponding settransientparent in Dialog should also be made behave the same) - Marco Martin On March 11, 2014, 3:52 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116722/ > ----------------------------------------------------------- > > (Updated March 11, 2014, 3:52 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > Allow setTransientParent from an item > > Because QWindow is not exposed as a property of Item in certain cases we > may not have access to the owning window in a qml that has menus. For > example WidgetExplorer. > > This patch also fixes crash in setTransientParent is called before a window > is set on > the menu. > > > Diffs > ----- > > src/declarativeimports/plasmacomponents/qmenu.h d1a5b73 > src/declarativeimports/plasmacomponents/qmenu.cpp a9b41a9 > > Diff: https://git.reviewboard.kde.org/r/116722/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel