----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119053/#review61344 -----------------------------------------------------------
Ship it! Push it, after looking at my small optimization suggestion. autotests/kglobalshortcuttest.cpp <https://git.reviewboard.kde.org/r/119053/#comment42723> (I can confirm that the kdelibs4 unittest passes with m_actionA here instead of m_actionB -- well, it obviously passes for both actions) src/kglobalaccel.cpp <https://git.reviewboard.kde.org/r/119053/#comment42722> sounds like shortcutFromInList(keys) should be put in a local variable, for use in this line and the next. - David Faure On July 1, 2014, 7:59 a.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119053/ > ----------------------------------------------------------- > > (Updated July 1, 2014, 7:59 a.m.) > > > Review request for KDE Frameworks, Plasma and David Faure. > > > Repository: kglobalaccel > > > Description > ------- > > Sorry for the last minute changes. I expected the bugs to be in the KCM and > not in kglobalaccel and unfortunately the autotests cannot be executed on > build.kde.org thus I never noticed that they don't pass :-( > > > Update actionShortcuts on auto loading shortcuts > > When using auto-loading the shortcut needs to be updated from what the > daemon returns. This had not been done yet and KGlobalAccel still kept > the shortcut passed in. > > With this change the autotests pass and the global shortcut KCM shows > the used shortcuts properly. > > Update actionShortcuts when daemon emits yourShortcutGotChanged > > Without updating the shortcut kept in actionShortcuts KGlobalAccel > still returns the old shortcut when invoking the shortcut() method. > > The unit test for this was unfortunately broken as it checked the > wrong QAction. This is also fixed with this change. > > Set componentName and DisplayName on QActions in KGlobalShortcutTest > > This fixes the testFindActionByKey. > > > Diffs > ----- > > autotests/kglobalshortcuttest.cpp 3f76760cb9e06a19d636a81546706cee548e1869 > src/kglobalaccel.cpp a98891c806764132b4f02b828d59584a85745fb3 > > Diff: https://git.reviewboard.kde.org/r/119053/diff/ > > > Testing > ------- > > unit tests pass now. > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel