> On April 9, 2014, 8:46 a.m., Vishesh Handa wrote: > > I'm a little hesitant about this because the user may never see emails as > > KMail only shows the first 8 results or so. The user can scroll but by > > default they will never see the Emails. > > > > Do you think it would make sense interleaving the email results with the > > file ones?
s/KMail/Krunner/ - Vishesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117441/#review55262 ----------------------------------------------------------- On April 9, 2014, 3:39 a.m., Eike Hein wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117441/ > ----------------------------------------------------------- > > (Updated April 9, 2014, 3:39 a.m.) > > > Review request for Baloo and Vishesh Handa. > > > Repository: baloo > > > Description > ------- > > Raising the ceiling would make the Baloo runner more useful in KRunner > frontends like Homerun and Homerun Kicker. > > While it might make sense for Homerun to use Baloo directly, this: > > a) Isn't currently possible in time for SC 4.13 to be released. > > b) Doesn't prevent users from using either the "KRunner" source in Homerun, > or the "Runner: Desktop Search" source. The former is the only way to get a > combined listing with other KRunner plugins, which might be desirable for > some users. > > c) Some users already have the above in their configs, and Baloo is the > natural upgrade path for them (and in fact, the just-released Homerun 1.2.3 > does this automatically). It'd nice if installing SC 4.13 doesn't degrade > their experience -- and 10 was the ceiling in the old nepomuksearch runner. > > It would be awesome if we could get this in before today's 4.13 tag. > > > Diffs > ----- > > src/runner/baloosearchrunner.cpp b62cb8b > > Diff: https://git.reviewboard.kde.org/r/117441/diff/ > > > Testing > ------- > > > Thanks, > > Eike Hein > >
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
