<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: VDG, Plasma.<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/D24844">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This action suffers from the following problems:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">It's buggy: tt does not take into account whether or not the maximum volume has already been raised globally using the settings window, and if it has and you toggle it, the slider's position gets stuck in an incorrect state</li>
<li class="remarkup-list-item">It's unnecessary: there is already a more reliable and useful way to raise the maximum volume level in the settings window</li>
<li class="remarkup-list-item">It't one of the last two items in the hamburger menu that user feedback indicates nobody clicks on, and we want to move in the direction of removing it</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>The hamburger menu no longer lists a "Raise Maximum Volume" menu item</p>

<p>You can still raise the baximum volume by clicking on the configure button in the applet to show the settings window</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>single-way-to-raise-max-volume (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24844">https://phabricator.kde.org/D24844</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applet/contents/ui/ListItemBase.qml</div></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>