Git commit 7bd7f38400b953a988a2cc942a518339e0b094a8 by Nate Graham.
Committed on 23/12/2018 at 04:14.
Pushed by ngraham into branch 'master'.

[KRun] when asked to open link in external browser, fall back to
mimeapps.list if nothing is set in kdeglobals

Summary:
Right now, when KRun is invoked to open an `http` or `https` link in a browser, 
it checks the `BrowserApplication` key in `~/.config/kdeglobals`. If nothing is 
set there (which is the default), then it introspects the link and figures out 
for itself what app to open, which is slow and can cause problems with certain 
links (see CCBUGs below).

This patch improves the browser discovery logic by additionally looking for a 
default browser in `~/.config/mimeapps.list`, which is the XDG file and it's 
where browsers set themselves as the default. So if there is a default browser 
set in there, KRun will consume that information immediately instead of doing 
the time-consuming and possibly error-inducing link introspection round-trip.
Related: bug 347870

Test Plan:
1. Open System Settings > Applications > Default Applications > Browser and 
click "In an application based on the contents of the url" (which is the 
default setting, but you might have changed it)
2. Set `BrowserApplication[$e]=` in `~/.config/kdeglobals`
3. Ensure that `~/.config/mimeapps/list` has a default browser set
4. Open any KDE app > Help menu > About KDE > Click on one of the links in the 
dialog

Without this patch, a KRun job is spawned that shows up in the
notification widget and the link may take a second or two to open in
your default browser.

With this patch, the link instantly opens in the browser.

Reviewers: #frameworks, broulik, cfeck, elvisangelaccio, dfaure

Reviewed By: dfaure

Subscribers: dfaure, rdieter, achauvel, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D17371

M  +13   -0    src/widgets/krun.cpp

https://commits.kde.org/kio/7bd7f38400b953a988a2cc942a518339e0b094a8

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/111032

Title:
  Usability glitches in default browser selection

To manage notifications about this bug go to:
https://bugs.launchpad.net/kdeadmin/+bug/111032/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to