Review Request 125339: Add an "Open" action

[Ragnar Thomsen]

> On Sept. 21, 2015, 10:57 p.m., Thomas Pfeiffer wrote:
> > No objections from the usability side.
> > About the quesiton what should be the default: Honestly, do we even need the preview mode at all? Why does Ark have to do things which other applications can already do?
> > My vote would be for killing the preview function alltogether and making "Open" the default. If the preview mode stays, it should stay as default, because nobody would use it if it wasn't the default action

Thanks for your comment. Regarding the Preview action, it opens a read-only viewer using KParts technology. Personally, I can't see the usefullness of it, but maybe some users like it. Arguments in favor of keeping it would be that it might be faster than opening a full external application and that it uses KDE-unique technology (KParts). Not sure if the latter is even a valid argument. I have taken a look at three other file archivers (Winrar, 7-Zip for Windows and GNOME's File-roller). Of those, only Winrar provides an integrated text viewer, while the other two just open the file using the associated app. The internal previewer in Ark is also broken for some mimetypes such as XML (see bug 201162) and adds dependency on KF5KHtml framework which is in Porting Aids (meaning we have to port it to something else at some point).

Do you have any opinion on what the names should be for "Open archive" and "Open file" actions (both are currently named just "Open")?


- Ragnar


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


On Sept. 21, 2015, 10:15 p.m., Ragnar Thomsen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125339/
> -----------------------------------------------------------
> 
> (Updated Sept. 21, 2015, 10:15 p.m.)
> 
> 
> Review request for KDE Utils, KDE Usability, Elvis Angelaccio, and Raphael Kubo da Costa.
> 
> 
> Bugs: 201162
>     http://bugs.kde.org/show_bug.cgi?id=201162
> 
> 
> Repository: ark
> 
> 
> Description
> -------
> 
> Add an Open action, which opens an archive entry with the associated application. This was previously only possible through the "Open With" action, but this required several clicks. Currently, the Preview action is still the default, i.e. when the user clicks an archive entry the Preview action is called.
> 
> QSignalMapper is used to connect the signals from the Open, Open With and Preview actions to the same slot.
> 
> As for Preview, the file is extracted to a QTemporaryDir and then opened. The temporarily extracted file is monitored for changes using QFileSystemWatcher and on change a KMessageBox is opened to query the user if the archive should be updated with the modified file. If the user accepts slotAddFiles() is called.
> 
> The previously unimplemented path argument of slotAddFiles() is used to allow for updating a file in a subdirectory within the archive.
> 
> Things to consider:
> 1. Should the Open action be the default e.g. when clicking an entry, instead of Preview?
> 2. Currently, the action has the same visible name as the action for opening archives in the File menu. One of them should be renamed. Which one, and to what?
> 3. Should we set some keyboard shortcuts for "Open" and "Open With"? If yes, which?
> 
> 
> Diffs
> -----
> 
>   part/ark_part.rc da04d47 
>   part/part.h 1d733c0 
>   part/part.cpp 8854fdd 
> 
> Diff: https://git.reviewboard.kde.org/r/125339/diff/
> 
> 
> Testing
> -------
> 
> Tested with tar-based, zip, rar and 7z archives.
> 
> 
> File Attachments
> ----------------
> 
> Context menu
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/21/a50431b5-34d5-4831-820a-82b174b4f243__open-1.png
> Action menu
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/21/dd6f7cbe-b610-4088-8586-4a76b68b6ef4__open-2.png
> 
> 
> Thanks,
> 
> Ragnar Thomsen
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20150922/ece640c0/attachment.html>