----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108305/#review25263 -----------------------------------------------------------
This review has been submitted with commit 7e0c1a487ad29e30c79dc8b7a17a210cf63c244a by Vishesh Handa to branch KDE/4.10. - Commit Hook On Jan. 9, 2013, 6 p.m., Vishesh Handa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108305/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2013, 6 p.m.) > > > Review request for Dolphin, Nepomuk and Plasma. > > > Description > ------- > > NepomukRunner: Redirect the 'nepomuk:/res/uuid' urls before opening them > > The nepomuksearch kioslave has a special syntax which results in the the > searched term being displayed in "human terms" without the need for > stating the url. The nepomuk kioslave does not have this feature. It, > however, uses the nepomuksearch kioslave internally. > > Applications like Dolphin often do not stat a url before displaying it > in the title bar. This results in users seing an ugly "nepomuk - res - > uuid" in the title bar instead of the proper display name as set in the > UDSEntry. > > In order to avoid this ugly title bar, we manually redirect all nepomuk > urls to nepomuksearch before they are opened by the runner. > > It's a hack, but it works! > > BUG: 287259 > > @Dolphin people: I just thought you might want to know that this is something > that will have to be fixed. > > > This addresses bug 287259. > http://bugs.kde.org/show_bug.cgi?id=287259 > > > Diffs > ----- > > plasma/generic/runners/nepomuksearch/nepomuksearchrunner.cpp 8075a6b > > Diff: http://git.reviewboard.kde.org/r/108305/diff/ > > > Testing > ------- > > Yup > > > Thanks, > > Vishesh Handa > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
