This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117420/

On April 13th, 2014, 4 a.m. EST, Vishesh Handa wrote:

src/file/kcm/folderselectionwidget.cpp (Diff revisions 2 - 3)
295
        return "[~]/" + url.mid(QDir::homePath().length() + 1);
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

On April 13th, 2014, 8 a.m. EST, 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


On April 13th, 2014, 9:01 a.m. EST, Lindsay Mathieson wrote:

Review request for Baloo.
By Lindsay Mathieson.

Updated April 13, 2014, 9:01 a.m.

Bugs: 333155
Repository: baloo

Description

Displays full path of excluded directories - reduce possibilities of confusion and ambiguity.

Testing

Build, installed - opens KCM module, full paths displayed, with ample room.

Diffs

  • src/file/kcm/folderselectionwidget.h (5e4e661)
  • src/file/kcm/folderselectionwidget.cpp (bc87a33)

View Diff