[Nepomuk] Review Request 117420: Fixes issue with full path of excluded directory not being shown

[Wyatt Epp]

> On April 7, 2014, 1:23 p.m., Vishesh Handa wrote:
> > I'm not inclined to accept this patch, you know my rationale. If you really want this then please only show the full path when there are duplicates.
> > 
> > Also, please contact the usability team and run it by them.
> 
> Lindsay Mathieson wrote:
>     I thought you were going to ask them about, but ok - will do.
>     
>     "If you really want this then please only show the full path when there are duplicates."
>     
>     Even more inconsistent.

Unfortunately paths are the most readily-available _human readable_ unique identifier that current filesystems have for preventing filename collisions, and the high incidence of duplication in file naming makes it abundantly clear that users rely on this.  On the other hand, showing the full path is kind of ugly and the repeating pattern of the prefix part makes it much more likely for users to skim and miss something important that was unintended. (I haven't seen Vishesh's rationale anywhere, but I'm guessing it's something in that vein?)

A possible compromise: show the path relative to the user's home.  This will generally distill it down to the specifier that users are familiar with in their own usage.  One class of potential degenerate cases concerns paths outside of home, such as devices mounted in [/run]/media.  For this, I beleive it would work to instead display relative to the mount point as long as it's clearly denoted in a human-readable form (e.g. device label).

Regards,
Wyatt


- Wyatt


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


On April 7, 2014, 11:59 a.m., Lindsay Mathieson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117420/
> -----------------------------------------------------------
> 
> (Updated April 7, 2014, 11:59 a.m.)
> 
> 
> Review request for Baloo and Nepomuk.
> 
> 
> 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.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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20140407/12b21579/attachment.html>