<table><tr><td style="">kishoreg created this revision.<br />kishoreg added reviewers: ngraham, romangg.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />kishoreg 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/D23768">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Recently, I submitted this patch (<a href="https://phabricator.kde.org/D23082" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D23082</a>) which had a side-effect which I hadn't noticed at the time: If the user-configured maximum volume for the applet was less than 100%, it would also use this as the maximum value on the sliders for all streams.</p>

<p>Before this patch:<br />
Devices tab: <a href="https://phabricator.kde.org/F7326896" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7326896: Screenshot_20190907_163232.png</a><br />
Applications tab: <a href="https://phabricator.kde.org/F7326898" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7326898: Screenshot_20190907_163203.png</a><br />
I have set the maximum volume in the applet settings to be 15%, and this setting is applied to all the sliders. Even the sliders which are intially at 100% get their maximum reset to 15% as soon as I interact with them. Note, for example, how even the microphone volume is capped at 15%.</p>

<p>After this patch:<br />
Devices tab: <a href="https://phabricator.kde.org/F7326904" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7326904: Screenshot_20190907_164055.png</a><br />
Applications tab: <a href="https://phabricator.kde.org/F7326905" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7326905: Screenshot_20190907_162913.png</a><br />
The maximum volume is still set to 15% in the applet settings, but this is does not affect the maximum values of the sliders for the device or application streams.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ol class="remarkup-list">
<li class="remarkup-list-item">Right-click on applet > 'Configure audio</li>
<li class="remarkup-list-item">Try to set maximum volume < 100% and click 'OK'.</li>
<li class="remarkup-list-item">Scroll on the systray icon to change maximum volume. Check if it is limited to the configured maximum.</li>
<li class="remarkup-list-item">Open the applet and check that all the streams are not limited to the value to the value the user configured in the applet.</li>
</ol>

</div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23768">https://phabricator.kde.org/D23768</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applet/contents/ui/ListItemBase.qml</div></div></div><br /><div><strong>To: </strong>kishoreg, ngraham, romangg<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>