D24844: [Applet] Remove "Raise Maximum Volume" menu item from Hamburger menu

Nathaniel Graham noreply at phabricator.kde.org
Mon Oct 21 22:41:48 BST 2019


ngraham created this revision.
ngraham added reviewers: VDG, Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  This action suffers from the following problems:
  
  - 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
  - It's unnecessary: there is already a more reliable and useful way to raise the maximum volume level in the settings window
  - 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

TEST PLAN
  The hamburger menu no longer lists a "Raise Maximum Volume" menu item
  
  You can still raise the baximum volume by clicking on the configure button in the applet to show the settings window

REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  single-way-to-raise-max-volume (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D24844

AFFECTED FILES
  applet/contents/ui/ListItemBase.qml

To: ngraham, #vdg, #plasma
Cc: 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191021/136ed621/attachment-0001.html>


More information about the Plasma-devel mailing list