<table><tr><td style="">nicolasfella created this revision.<br />nicolasfella added reviewers: Plasma, drosca.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />nicolasfella 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/D23620">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Initially no sink is selected in the combobox which leads to an empty speaker test widget. Furthermore, when only one sink is available the combobox isn't shown at all, resulting in the speaker test not working because currentIndex will be -1.</p>

<p>Fix this by setting the initial index to 0 and use 0 instead of -1 for querying model data when the combobox isn't shown</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>With 1 available sink speaker test works. With two available sinks the first one is selected at startup, speaker test works</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>BRANCH</strong><div><div>gix</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23620">https://phabricator.kde.org/D23620</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kcm/package/contents/ui/Advanced.qml</div></div></div><br /><div><strong>To: </strong>nicolasfella, Plasma, drosca<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>