Review Request 113191: Files passed as arguments: Ignore unsupported files and open archives as folders

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113191/#review41627
-----------------------------------------------------------


This review has been submitted with commit cd9e50ae4f3ded5a78d0cfb09a67684a9c15d726 by Frank Reininghaus on behalf of Erik Hahn to branch KDE/4.11.

- Commit Hook


On Oct. 10, 2013, 2:55 p.m., Erik Hahn wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113191/
> -----------------------------------------------------------
> 
> (Updated Oct. 10, 2013, 2:55 p.m.)
> 
> 
> Review request for Dolphin and Frank Reininghaus.
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> With this patch, Dolphin ignores all files passed to it that it can't handle instead of opening them externally. I assume that it wasn't actually intended to be usable as a xdg-open replacement.
> 
> Also, archives are now opened inside Dolphin so it can be used as an archive manager at least for local files. If the user tries to open a remote archive Dolphin still opens it externally; I have observed that if it receives one as an argument, it will display a pseudo-folder that contains only said archive. So having it set as the archive handler is still broken, but in a less annoying way.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinmainwindow.cpp 4128cdf 
> 
> Diff: http://git.reviewboard.kde.org/r/113191/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Erik Hahn
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20131013/5e134805/attachment.htm>