Bug 427297

Summary:	Name of remote pulseaudio output is not user-friendly in tooltip of plasma-pa applet
Product:	[Unmaintained] plasma-pa	Reporter:	Patrick Silva <bugseforuns>
Component:	applet	Assignee:	David Rosca <nowrep>
Status:	RESOLVED FIXED
Severity:	normal	CC:	nate, plasma-bugs-null
Priority:	NOR	Keywords:	junior-jobs
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	popup - user-friendly name for remote output tooltip - name for remote output is not user-friendly

Description Patrick Silva 2020-10-03 16:13:23 UTC

Created attachment 132093 [details]
popup - user-friendly name for remote output

SUMMARY
I stream audio, via local network, from my laptop to the speakers connected to my desktop computer.
See https://wiki.archlinux.org/index.php/PulseAudio#Networked_audio

STEPS TO REPRODUCE
1. stream audio via local network to remote pulseaudo server
2. hover over sound icon in the system tray
3. 

OBSERVED RESULT
As we can see in the attached screenshots, plasma-pa applet shows
user-friendly name for the remote pulseaudio output in its popup but it does not in its tooltip.

EXPECTED RESULT
name of remote pulseaudio output should be user-friendly in the tooltip
of plasma-pa applet too.

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.20.80
KDE Frameworks Version: 5.75.0
Qt Version: 5.15.0

Comment 1 Patrick Silva 2020-10-03 16:14:30 UTC

Created attachment 132094 [details]
tooltip - name for remote output is not user-friendly

Comment 2 Nate Graham 2020-10-05 21:03:07 UTC

The bug is somewhere in how the applet sets Plasmoid.toolTipSubText: in main.qml.

Either the port is getting set incorrectly such that port.description is set to the wrong string, or paSinkModel.preferredSink.name is set to the wrong string.

Probably a good junior job.

Comment 3 Patrick Silva 2021-05-01 12:32:39 UTC

Fixed on neon unstable. :)

https://invent.kde.org/plasma/plasma-pa/-/merge_requests/61