Review Request: Use user preferred application to preview files.

[Michal Sciubidlo]

> On Nov. 23, 2011, 10:25 p.m., Raphael Kubo da Costa wrote:
> > The patch as it stands ends up effectively disabling the preview dialog, which is a no-go -- calling KMimeTypeTrader::query() without specifying the service type will use "Application" as default and all files will be previewed externally (there is another bug for allowing a user to choose to view a file externally, but that's another issue).
> > 
> > The related bug is a tricky issue whose proper fix I still don't know -- see http://article.gmane.org/gmane.comp.kde.devel.core/59514
> 
> Michal Sciubidlo wrote:
>     Why disabling preview dialog is no-go?
>     Now ark ignores kde and user settings. It's coincidence that kde default app is konqueror. When kde will change it as you suggested kde.devel.core ark will ignore it.
> 
> Raphael Kubo da Costa wrote:
>     Disabling it is a no-go because it works most of the time and is a nice feature to have. Ark does not ignore KDE and user settings: the code first looks for read-only parts capable of displaying the given mimetype (which is defined by the user in System Settings -> File Associations -> Embedding) and then for an application which can do that (System Settings -> File Associations -> General), so if KDE/the user changes settings, those will be automatically be picked up by Ark (I mean, KMimeTypeTrader).
> 
> Michal Sciubidlo wrote:
>     I see where i was wrong. I still think that current solution is not good. There are others confused by this (two last comments in https://bugs.kde.org/show_bug.cgi?id=201162). Associations -> General are importand and Associations -> Embedding should be hidden (only for apps use or even deleted). IMO ideal solution would be using assoc->general list. If first element has read only part ark should use it. If it don't have it full app should be opened. Any later general associations and whole embedded list should be ingored.
>     
>     Two lists, apps can pick from any of them... that is "evil".

Do you think this solution will be ok or two lists are better and leave it?


- Michal


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


On Nov. 23, 2011, 7:13 p.m., Michal Sciubidlo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103221/
> -----------------------------------------------------------
> 
> (Updated Nov. 23, 2011, 7:13 p.m.)
> 
> 
> Review request for KDE Utils and Raphael Kubo da Costa.
> 
> 
> Description
> -------
> 
> Use user preferred application to preview files. For now user has to delete every read only kpart from list to use other application as defcault.
> 
> 
> This addresses bug 201162.
>     http://bugs.kde.org/show_bug.cgi?id=201162
> 
> 
> Diffs
> -----
> 
>   part/arkviewer.cpp 9bfd651 
> 
> Diff: http://git.reviewboard.kde.org/r/103221/diff/diff
> 
> 
> Testing
> -------
> 
> Set kate as default, rekonq as second. Result: file opened with kate.
> Set rekonq as default, kate as second. Result: file opened with rekonq.
> 
> 
> Thanks,
> 
> Michal Sciubidlo
> 
>

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