<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Plasma, cfeck.<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/D17181">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>BUG: 350663<br />
FIXED-IN: 5.12.8</p>

<p>When choosing a default browser app using the app list combobox, the choice was noly being saved after the combobox was used, because its signal was connected to an inline function that saved the choice, while the radio button that enabled the combobox was not. As a result, if you clicked on that radio button and didn't need to use the combobox because it was already showing your preferred browser, that action was not actually getting saved.</p>

<p>This patch fixes the situation ym moving the inline function into a real explicit slot function and connecting the radio button to it as well</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Go to the component chooser KCM's default browser module</li>
<li class="remarkup-list-item">Reset to defaults, then leave the KCM and open it again</li>
<li class="remarkup-list-item">Click the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">in the following application</span></span></span> radio button</li>
<li class="remarkup-list-item">Do NOT touch the combobox</li>
<li class="remarkup-list-item">Click <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Apply</span></span></span></li>
<li class="remarkup-list-item">Leave the KCM and come back</li>
</ul>

<p>With this patch, the radio button's state is preserved and the correct browser is selected in the combobox.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>fix-default-browser-app-combobox (branched from Plasma/5.12)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17181">https://phabricator.kde.org/D17181</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/componentchooser/componentchooserbrowser.cpp<br />
kcms/componentchooser/componentchooserbrowser.h</div></div></div><br /><div><strong>To: </strong>ngraham, Plasma, cfeck<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>