Do you still want this Vishesh? On 15 April 2014 07:10, Lindsay Mathieson <lindsay.mathie...@gmail.com> wrote: > Do you still want this Vishesh? > > On Sat, 12 Apr 2014 11:02:58 PM Lindsay Mathieson wrote: >> > On April 13, 2014, 4 a.m., Vishesh Handa wrote: >> > > src/file/kcm/folderselectionwidget.cpp, line 295 >> > > <https://git.reviewboard.kde.org/r/117420/diff/2-3/?file=263314#file2633 >> > > 14line295>> > >> > > I've given this more thought, and I dislike the '~/'. Please remove >> > > it. >> > > >> > > The rest is fine. >> > > >> > > I can ask for an exception from the release team and ship this with >> > > 4.13 >> > >> > Lindsay Mathieson wrote: >> > Trouble is, if you then add $HOME as an exclude path it just displays >> > the Home Icon and no string at all. Would "HomeIcon/" be better? >> I've taken the liberty of modifying the iconName() func so that it returns >> the home icon or drive icon for any path starting with home or mount point >> respectively. Easy enough to undo if you do not like it. >> >> >> - Lindsay >> >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://git.reviewboard.kde.org/r/117420/#review55527 >> ----------------------------------------------------------- >> >> On April 13, 2014, 9:01 a.m., Lindsay Mathieson wrote: >> > ----------------------------------------------------------- >> > This is an automatically generated e-mail. To reply, visit: >> > https://git.reviewboard.kde.org/r/117420/ >> > ----------------------------------------------------------- >> > >> > (Updated April 13, 2014, 9:01 a.m.) >> > >> > >> > Review request for Baloo. >> > >> > >> > Bugs: 333155 >> > >> > http://bugs.kde.org/show_bug.cgi?id=333155 >> > >> > Repository: baloo >> > >> > >> > Description >> > ------- >> > >> > Displays full path of excluded directories - reduce possibilities of >> > confusion and ambiguity. >> > >> > >> > Diffs >> > ----- >> > >> > src/file/kcm/folderselectionwidget.h 5e4e661 >> > src/file/kcm/folderselectionwidget.cpp bc87a33 >> > >> > Diff: https://git.reviewboard.kde.org/r/117420/diff/ >> > >> > >> > Testing >> > ------- >> > >> > Build, installed - opens KCM module, full paths displayed, with ample >> > room. >> > >> > >> > Thanks, >> > >> > Lindsay Mathieson > > -- > Lindsay
-- Lindsay >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<