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

Erik Hahn erik_hahn at gmx.de
Thu Oct 10 19:34:57 BST 2013



> On Oct. 10, 2013, 3:47 p.m., Frank Reininghaus wrote:
> > Thanks for the new patch, looks good! I think that this can go into the KDE/4.11 branch - it seems unlikely that this will cause a show-stopping regression.
> > 
> > Do you have a git account, or shall I push the commit for you?

Your welcome -- I have no git account so please push it for me.


- Erik


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


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/20131010/6383b1f7/attachment.htm>


More information about the kfm-devel mailing list