----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125489/#review86274 -----------------------------------------------------------
The patch breaks starting of the Presentation by pressing the Start Presentation button without extending the submenu. - Thorsten Zachmann On Oct. 2, 2015, 9:09 p.m., David Faure wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125489/ > ----------------------------------------------------------- > > (Updated Oct. 2, 2015, 9:09 p.m.) > > > Review request for Calligra and Thorsten Zachmann. > > > Repository: calligra > > > Description > ------- > > Cleanup KActionMenu usage: activated() is never emitted, member var not > needed. > > > Diffs > ----- > > stage/part/KPrView.h 1ca41cee7fc38cc2a3156fad1f2283eaf217e2ef > stage/part/KPrView.cpp 432bc5e023f41a8927cbe9e854b2205962ae4e4b > > Diff: https://git.reviewboard.kde.org/r/125489/diff/ > > > Testing > ------- > > No GUI changes seen, the start presentation actions still work > > > Thanks, > > David Faure > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
