<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Plasma, VDG.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />ngraham requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D17372">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>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.</p>

<p>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.</p>

<p>BUG: 100016<br />
FIXED-IN: 5.15.0</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Here's what it looks like now:</p>

<p>Verifications performed:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">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.</li>
<li class="remarkup-list-item">Changing the browser in the combobox correctly updates both <tt style="background: #ebebeb; font-size: 13px;">~/.config/kdeglobals</tt> and <tt style="background: #ebebeb; font-size: 13px;">~/.configmimeapps.list</tt></li>
<li class="remarkup-list-item">The combobox's default setting is now correctly identical to what <tt style="background: #ebebeb; font-size: 13px;">~/.config/mimeapps.list</tt> specifies, if anything</li>
<li class="remarkup-list-item">KRun still does unnecessary URL introspection when there is in fact a default browser set in <tt style="background: #ebebeb; font-size: 13px;">~/.config/mimeapps.list</tt> but ~/.config/kdeglobals` doesn't have a default browser set, which is fixed separately in KIO with <a href="https://phabricator.kde.org/D17371" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17371</a></li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>browser-chooser-service-and-exec-only (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17372">https://phabricator.kde.org/D17372</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/componentchooser/browserconfig_ui.ui<br />
kcms/componentchooser/componentchooserbrowser.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, Plasma, VDG<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>