Bug 506463 - Input Streams delegate is inappropriately indented when there is more than one Output Stream
Summary: Input Streams delegate is inappropriately indented when there is more than on...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Audio Volume widget (other bugs)
Version First Reported In: 6.4.1
Platform: Other Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2025-07-02 00:37 UTC by ratijas
Modified: 2025-08-23 21:38 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.4.5
Sentry Crash Report:


Attachments
Audio Volume - Applications - Input Streams with indentation (176.84 KB, image/png)
2025-07-02 00:37 UTC, ratijas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ratijas 2025-07-02 00:37:58 UTC
Created attachment 182846 [details]
Audio Volume - Applications - Input Streams with indentation

SUMMARY

See the screenshot.

STEPS TO REPRODUCE
1. Play two audio sources at once
2. Have one input/microphone active
3. Open the Audio Volume applet

OBSERVED RESULT
Most of Input Stream delegate's content except its leading icon suddenly gets indented for no reason.

EXPECTED RESULT


SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 6.4.1
KDE Frameworks Version: 6.15.0
Qt Version: 6.9.1
Kernel Version: 6.15.4-arch2-1 (64-bit)
Graphics Platform: X11
Processors: 16 × AMD Ryzen 9 7940HS w/ Radeon 780M Graphics
Memory: 64 GiB of RAM (54.7 GiB usable)
Graphics Processor 1: AMD Radeon RX 7700S
Graphics Processor 2: AMD Radeon 780M Graphics


ADDITIONAL INFORMATION
What has prompted a yet-another-redesign?
Comment 1 David Edmundson 2025-07-02 10:05:01 UTC
Found it:

We show buttons based 

visible: (item.type === "sink" || item.type === "source") && item.ListView.view.count > 1

We provide matching padding when the other view has buttons (via otherViewHasButtons) but that is based solely on
item.ListView.view.count > 1   and not checking the type.

>What has prompted a yet-another-redesign?
I cannot see any evidence of this being caused by a redesign.
Comment 2 Nate Graham 2025-08-22 21:19:05 UTC
Fixed by Christoph Wolk with https://invent.kde.org/plasma/plasma-pa/-/commit/a66ce3f64df762b5d92a4a23b2394abf0fa9a436 in Plasma 6.4.5!
Comment 3 cwo 2025-08-23 21:36:52 UTC
Git commit a66ce3f64df762b5d92a4a23b2394abf0fa9a436 by Christoph Wolk.
Committed on 22/08/2025 at 14:35.
Pushed by cwo into branch 'master'.

applet: don't indent streams

For sources, we add indentation to the entry if the other source
category has radio buttons but the current category does not (due to
having only a single entry), to make them align properly. The code also
applies this to streams, but this makes no sense as streams never have
radio buttons.

Instead, we apply the indentation only for devices, not streams. For
clarity, we factor this out as a property and use it for other
conditionals that test for being a stream, rather than duplicate the
code.
FIXED-IN: 6.4.5

M  +5    -4    applet/ListItemBase.qml

https://invent.kde.org/plasma/plasma-pa/-/commit/a66ce3f64df762b5d92a4a23b2394abf0fa9a436
Comment 4 cwo 2025-08-23 21:38:44 UTC
Git commit cab7d0d8c62da8ffe4067c6a81a4febfe2d0c3b2 by Christoph Wolk.
Committed on 22/08/2025 at 14:56.
Pushed by cwo into branch 'Plasma/6.4'.

applet: don't indent streams

For sources, we add indentation to the entry if the other source
category has radio buttons but the current category does not (due to
having only a single entry), to make them align properly. The code also
applies this to streams, but this makes no sense as streams never have
radio buttons.

Instead, we apply the indentation only for devices, not streams. For
clarity, we factor this out as a property and use it for other
conditionals that test for being a stream, rather than duplicate the
code.
FIXED-IN: 6.4.5


(cherry picked from commit a66ce3f64df762b5d92a4a23b2394abf0fa9a436)

Co-authored-by: Christoph Wolk <cwo.kde@posteo.net>

M  +5    -4    applet/ListItemBase.qml

https://invent.kde.org/plasma/plasma-pa/-/commit/cab7d0d8c62da8ffe4067c6a81a4febfe2d0c3b2