search

lxqt / pavucontrol-qt

A Pulseaudio mixer in Qt (port of pavucontrol)
https://lxqt.github.io
GNU General Public License v2.0
114 stars 28 forks source link

Typos only? #254

Open stefonarch opened 1 year ago

stefonarch commented 1 year ago

https://github.com/lxqt/pavucontrol-qt/blob/55bcfc9826efd7bd4c352e65db8ae6b32149094c/src/pavucontrol.cc#L622

I stumbled opon those while updating translations, it is new but probably nobody has updated translations for a long time, it has 3 typos and I'm not sure if this text is really ok. The command mentioned, and "root window" sounds strange to me but pulseaudio is not my thing.

stefonarch commented 1 year ago

@palinek any clue about that text?

stefonarch commented 3 weeks ago

Connection to PulseAudio failed. Automatic retry in 5s.

In this case this is likely because PULSE_SERVER in the Environment/X11 Root Window Propertiesor default-server in client.conf is misconfigured.
This situation can also arrise when PulseAudio crashed and left stale details in the X11 Root Window.
If this is the case, then PulseAudio should autospawn again, or if this is not configured you shouldrun start-pulseaudio-x11 manually.

I think we should drop all this text after "retry in 5s". 'start-pulseaudio-x11 ' doesn't exist anywhere.

tsujan commented 3 weeks ago

I agree. At best it's confusing, but it may also give wrong info.

palinek commented 3 weeks ago

It is the same text as present in original pavucontrol and is there from day-1 after PCMan made the port to Qt. It would be probably better to raise the issue upstream, wait and follow for their decision.