Review Request 120568: Save the default browser into the group [Default Applications]

David Faure faure at kde.org
Sun Oct 12 20:51:50 UTC 2014


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



kcms/componentchooser/componentchooserbrowser.cpp
<https://git.reviewboard.kde.org/r/120568/#comment47598>

    No, I am very much against this.
    
    The whole point of "use the right KDE app" is to use the right KDE app, not to fire up kfmclient_html for everything including text files, images, calligra documents etc.
    The "html" suffix is especially wrong since it would try to use khtmlpart or webkitpart for all of this (explicit mimetype on the command-line).
    At least using kde-open would work for all mimetypes, but this is slower (one more intermediate process) than letting the calling app use KRun directly.
    
    I know that gnome saves the webbrowser as x-scheme-handler/http, but I have always considered this a mistake (and we discussed it at length on the xdg list, so the status quo will remain).
    
    One solution would be to keep this code (with kde-open) but then special case "if the http scheme handler is kde-open then use KRun directly" in the kio code that looks up x-scheme-handler stuff (grep says: KRun and DesktopExecParser).


- David Faure


On Oct. 12, 2014, 8:40 p.m., Luc Menut wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120568/
> -----------------------------------------------------------
> 
> (Updated Oct. 12, 2014, 8:40 p.m.)
> 
> 
> Review request for Plasma and David Faure.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> Save the default browser by writing x-scheme-handler/http and x-scheme-handler/https into the group [Default Applications] in the file mimeapps.list .
> Nowadays, many applications look at user preferences for x-scheme-handler/http to determine the default browser, so setting this value would increase interoperability with these applications.
> 
> regards,
> 
> Luc Menut - Mageia
> 
> PS: I don't have write access to kde git, so could you commit the change if the patch looks fine. Thanks.
> 
> 
> Diffs
> -----
> 
>   kcms/componentchooser/componentchooserbrowser.cpp 61af1fd 
> 
> Diff: https://git.reviewboard.kde.org/r/120568/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Luc Menut
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20141012/7b2d5953/attachment-0001.html>


More information about the Plasma-devel mailing list