D17372: [componentchooser KCM] Make KIO browser option the fallback only and remove from the UI

Nathaniel Graham noreply at phabricator.kde.org
Wed Dec 5 19:57:16 GMT 2018


ngraham created this revision.
ngraham added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  Right now the componentchooser KCM's browser module has a really confusing setting ("In an application based on the contents of the url") that's the default. There are ancient Buzailla tickets about this being confusing, and some users report that it actually does the wrong thing for certain URLs.
  
  This patch removes the UI for this feature and makes it a hidden fallback-only behavior for the rare case that there aren't any web browsers installed.
  
  BUG: 100016
  FIXED-IN: 5.15.0

TEST PLAN
  Here's what it looks like now:
  
  Verifications performed:
  
  - There are no actual settings changes for any users, since browsers ask to set themselves as the default, and distros ship config files already specifying their preferred browser.
  - Changing the browser in the combobox correctly updates both `~/.config/kdeglobals` and `~/.configmimeapps.list`
  - The combobox's default setting is now correctly identical to what `~/.config/mimeapps.list` specifies, if anything
  - KRun still does unnecessary URL introspection when there is in fact a default browser set in `~/.config/mimeapps.list` but ~/.config/kdeglobals` doesn't have a default browser set, which is fixed separately in KIO with D17371 <https://phabricator.kde.org/D17371>

REPOSITORY
  R119 Plasma Desktop

BRANCH
  browser-chooser-service-and-exec-only (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D17372

AFFECTED FILES
  kcms/componentchooser/browserconfig_ui.ui
  kcms/componentchooser/componentchooserbrowser.cpp

To: ngraham, #plasma, #vdg
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181205/6e1ab5ac/attachment.html>


More information about the Plasma-devel mailing list