----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106068/#review17871 -----------------------------------------------------------
Ship it! i am using it since a while and doesn't seem to be problems. only one suggestion in the behavior: right now clicking on a result doesn't do anything (ie doesn't have run()). it may copy the definition in the clipboard, so is easy accessible - Marco Martin On Aug. 18, 2012, 3:57 a.m., Jason A. Donenfeld wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106068/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2012, 3:57 a.m.) > > > Review request for Plasma, Jason A. Donenfeld and Aaron J. Seigo. > > > Description > ------- > > Jason's dictionary runner plugin for krunner. > > > Diffs > ----- > > runners/CMakeLists.txt f487563c58b5bacc5e49b9e1a0f6e306956bbf7f > runners/dictionary/CMakeLists.txt PRE-CREATION > runners/dictionary/Messages.sh PRE-CREATION > runners/dictionary/dictionarymatchengine.h PRE-CREATION > runners/dictionary/dictionarymatchengine.cpp PRE-CREATION > runners/dictionary/dictionaryrunner.h PRE-CREATION > runners/dictionary/dictionaryrunner.cpp PRE-CREATION > runners/dictionary/dictionaryrunner_config.h PRE-CREATION > runners/dictionary/dictionaryrunner_config.cpp PRE-CREATION > runners/dictionary/plasma-runner-dictionary.desktop PRE-CREATION > runners/dictionary/plasma-runner-dictionary_config.desktop PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/106068/diff/ > > > Testing > ------- > > Works well under various different loads. > > > Screenshots > ----------- > > The Runner in Action > http://git.reviewboard.kde.org/r/106068/s/679/ > > > Thanks, > > Jason A. Donenfeld > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
