----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108700/#review26519 -----------------------------------------------------------
one small issue to resolve, and then it can go in... plasma/declarativeimports/plasmacomponents/qmenu.cpp <http://git.reviewboard.kde.org/r/108700/#comment20147> we try not to use slot* names (as they tend to drift over time from the signals connected and/or other uses), but rather state what the method does. so in this case perhaps "menuClosing()" or sth - Aaron J. Seigo On Feb. 1, 2013, 3:04 p.m., Aurélien Gâteau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108700/ > ----------------------------------------------------------- > > (Updated Feb. 1, 2013, 3:04 p.m.) > > > Review request for Plasma. > > > Description > ------- > > Make sure Menu.status is set to Closed when the menu closes itself > > > Diffs > ----- > > plasma/declarativeimports/plasmacomponents/qmenu.h > 00b73e47f1bbfe12aa1f6d5f7dfc8de7f7a0dfe1 > plasma/declarativeimports/plasmacomponents/qmenu.cpp > e91c365e401d751d42430eca36dea42f1258ee50 > > Diff: http://git.reviewboard.kde.org/r/108700/diff/ > > > Testing > ------- > > Run attached testmenu.qml program. Without the patch, the text says "Opened" > even after closing the menu. > > > File Attachments > ---------------- > > Test program > http://git.reviewboard.kde.org/media/uploaded/files/2013/02/01/testmenu.qml > > > Thanks, > > Aurélien Gâteau > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
