D27874: Disable the audio stream indicator for IOTM by default

2020-05-19 Thread Noah Davis
ndavis abandoned this revision. ndavis added a comment. Abandoning this since nobody wanted it. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27874 To: ndavis, #vdg, #plasma Cc: gvgeo, ngraham, broulik, cblack, plasma-devel, Orage, LeGast00n, The-Feren-OS-D

D27874: Disable the audio stream indicator for IOTM by default

2020-03-16 Thread Nathaniel Graham
ngraham added a comment. I won't block this, but personally I don't find the indicator to be a problem with my IOTM. When I click on an icon, I'm almost always clicking on a pixel adjacent to the screen edge, which is on the other side of where the audio indicator is drawn, so it never gets

D27874: Disable the audio stream indicator for IOTM by default

2020-03-16 Thread Noah Davis
ndavis added a comment. In D27874#628714 , @ngraham wrote: > Title needs updating to reflect the current state of the patch, if you'd still like to propose that we do this. I'm still not sure I understand the point. The indicator is already disa

D27874: Disable the audio stream indicator for IOTM

2020-03-16 Thread Nathaniel Graham
ngraham added a comment. Title needs updating to reflect the current state of the patch, if you'd still like to propose that we do this. I'm still not sure I understand the point. The indicator is already disable-able for both the IOTM and TTM. What value is being gained here? REPOSITORY

D27874: Disable the audio stream indicator for IOTM

2020-03-05 Thread Noah Davis
ndavis updated this revision to Diff 77045. ndavis added a comment. - Use an option instead REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27874?vs=77043&id=77045 BRANCH disable-audioindicator-iotm (branched from master) REVISION DETAIL https:/

D27874: Disable the audio stream indicator for IOTM

2020-03-05 Thread Noah Davis
ndavis added a comment. Do we really want a feature that new users have to learn to dodge? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27874 To: ndavis, #vdg, #plasma Cc: ngraham, broulik, cblack, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh

D27874: Disable the audio stream indicator for IOTM

2020-03-05 Thread George Vogiatzis
gvgeo added a comment. Disable the mute function then. And maybe add as an option. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27874 To: ndavis, #vdg, #plasma Cc: gvgeo, ngraham, broulik, cblack, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh,

D27874: Disable the audio stream indicator for IOTM

2020-03-05 Thread Noah Davis
ndavis updated this revision to Diff 77043. ndavis added a comment. - Hide audio indicator setting for IOTM REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27874?vs=77040&id=77043 BRANCH disable-audioindicator-iotm (branched from master) REVISION

D27874: Disable the audio stream indicator for IOTM

2020-03-05 Thread Nathaniel Graham
ngraham added a comment. I can't say that it's gotten in the way for me either, with my thick panel. It does overlap the badge icon though which is a bit annoying. Also yes, if you want to remove this from the IOTM, you need to hide the setting on the IOTM's config page too. REPOSITORY

D27874: Disable the audio stream indicator for IOTM

2020-03-05 Thread Kai Uwe Broulik
broulik added a comment. -1 to removing this feature. You also need to remove the checkbox on the settings page then. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27874 To: ndavis, #vdg, #plasma Cc: broulik, cblack, plasma-devel, Orage, LeGast00n, The-

D27874: Disable the audio stream indicator for IOTM

2020-03-05 Thread Carson Black
cblack added a comment. -1, the badge hasn't gotten in my way since the first week or so using it, despite the small size of my panel. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D27874 To: ndavis, #vdg, #plasma Cc: cblack, plasma-devel, Orage, LeGast00n,

D27874: Disable the audio stream indicator for IOTM

2020-03-05 Thread Noah Davis
ndavis created this revision. ndavis added reviewers: VDG, Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. ndavis requested review of this revision. REVISION SUMMARY After having used it for months, it still gets in the way when used with the Icons-only Task Man