Bug 495752

Summary: Audio device test dialog is small when Pro Audio is selected
Product: [Applications] systemsettings Reporter: Carlos Cañellas <ratiosu.cct>
Component: kcm_pulseaudioAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: isma.af, nate
Priority: NOR    
Version: 6.2.2   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed In: 6.2.4
Sentry Crash Report:
Attachments: Seems smol

Description Carlos Cañellas 2024-11-03 15:03:38 UTC
Created attachment 175488 [details]
Seems smol

SUMMARY


STEPS TO REPRODUCE
1. Select Pro Audio in the dropdown of one device.
2. Try to test audio clicking on Test

OBSERVED RESULT


EXPECTED RESULT
A bigger dialogue, perhaps more space for the title of the 

SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 41
KDE Plasma Version: 6.2.2
KDE Frameworks Version: 6.7.0
Qt Version: 6.8.0
Kernel Version: 6.11.5-300.fc41.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 20 × 12th Gen Intel® Core™ i7-1280P
Memory: 30.9 GiB of RAM
Graphics Processor: Mesa Intel® Graphics

ADDITIONAL INFORMATION
My laptop is a Dell XPS Plus 13 9320
Comment 1 Bug Janitor Service 2024-11-05 20:52:08 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-pa/-/merge_requests/295
Comment 2 Ismael Asensio 2024-11-06 17:28:24 UTC
Git commit 36075f9e5f8d58ad2c687b91ec9a9cbef8c59fb3 by Ismael Asensio.
Committed on 05/11/2024 at 20:52.
Pushed by iasensio into branch 'master'.

kcm: Fix speaker test layout for Pro-Audio profile

Known audio channels (like 'front-right', `rear-left`) are assigned
rows 0 to 2 in the Grid layout so they are placed in their spatial
position around the user's avatar

Unknown channels (those produced by the Pro Audio pipewire profile)
have no spatial info and are assigned to rows 3 and following

The spacer items were also assigned to row 3, which collided with
those, breaking the layout. Let's give the spacers a high enough
row number so there is no collision with an already occupied cell
FIXED-IN: 6.2.4

M  +2    -2    src/kcm/ui/main.qml

https://invent.kde.org/plasma/plasma-pa/-/commit/36075f9e5f8d58ad2c687b91ec9a9cbef8c59fb3
Comment 3 Ismael Asensio 2024-11-06 17:34:51 UTC
Git commit 0b4a26ada8258f4d25fa5069f239f1041eefecb5 by Ismael Asensio.
Committed on 06/11/2024 at 17:31.
Pushed by iasensio into branch 'Plasma/6.2'.

kcm: Fix speaker test layout for Pro-Audio profile

Known audio channels (like 'front-right', `rear-left`) are assigned
rows 0 to 2 in the Grid layout so they are placed in their spatial
position around the user's avatar

Unknown channels (those produced by the Pro Audio pipewire profile)
have no spatial info and are assigned to rows 3 and following

The spacer items were also assigned to row 3, which collided with
those, breaking the layout. Let's give the spacers a high enough
row number so there is no collision with an already occupied cell
FIXED-IN: 6.2.4


(cherry picked from commit 36075f9e5f8d58ad2c687b91ec9a9cbef8c59fb3)

Co-authored-by: Ismael Asensio <isma.af@gmail.com>

M  +2    -2    src/kcm/ui/main.qml

https://invent.kde.org/plasma/plasma-pa/-/commit/0b4a26ada8258f4d25fa5069f239f1041eefecb5