<table><tr><td style="">gvgeo planned changes to this revision.<br />gvgeo added a comment.
</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/D26234">View Revision</a></tr></table><br /><div><div><p>Either I missing something, or I need to make it more clear.</p>

<p>With this patch, the maximum volume setting is for the devices/apps that have enabled the raise maximum option.<br />
By default users will be limited to 100. Will need to enable raise maximum volume for some application or device first, for the default value of 150 to have any effect.<br />
Even then, the rest devices and applications will keep using 100 as maximum value.</p>

<p>This could take more fine tuning, but was trying to keep patches small and clean:<br />
renaming 'maximum volume' to 'custom maximum volume'(or something better to explain the difference),<br />
adding an 'raise maximum volume for all devices/applications' option,<br />
make label change to 'lower maximum volume' when lower than 100% is selected.</p>

<p>I cannot see any solution that will have as default 100 and not complicate the raise maximum volume even more.<br />
In any mode(separate or global), if default maximum volume will be kept 100, user will need to enable raise maximum volume and change the maximum volume.<br />
Even more important in case of removal of the setting. Will need to predefined a value.<br />
Right now, the slider already uses 150 which users cannot be change, does not cause any issue(except the existence of two modes).</p>

<p>Marking as planned changes, no need to appear in review queues, if global mode is the way forward.</p></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/D26234">https://phabricator.kde.org/D26234</a></div></div><br /><div><strong>To: </strong>gvgeo, VDG, Plasma, drosca<br /><strong>Cc: </strong>anthonyfieroni, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>