[GH-ISSUE #81] Sound stops working on remote machine #2946

Closed
opened 2026-03-13 00:24:56 +03:00 by kerem · 5 comments
Owner

Originally created by @majkinetor on GitHub (Feb 8, 2021).
Original GitHub issue: https://github.com/1Remote/1Remote/issues/81

Originally assigned to: @VShawn on GitHub.

Describe the bug
I set up connection to my media center on Windows and as soon as I connect sound stops working.

The connection settings

image

The native RD seting

image

It looks like PRM has only 2 settings (play on host/do not play) while it needs 3.

Expected behavior
The sound shouldn't stop playing on remote machine, just like it doesn't do on native RD when above option is set.

Desktop (please complete the following information):

  • OS: Win10 10.0.19042
  • PRemoteM Version 0.5.8.2101192131
Originally created by @majkinetor on GitHub (Feb 8, 2021). Original GitHub issue: https://github.com/1Remote/1Remote/issues/81 Originally assigned to: @VShawn on GitHub. **Describe the bug** I set up connection to my media center on Windows and as soon as I connect sound stops working. <details><summary>The connection settings</summary> ![image](https://user-images.githubusercontent.com/85767/107262598-05f08780-6a41-11eb-97f9-16474e84e248.png)</details> <details><summary>The native RD seting </summary> ![image](https://user-images.githubusercontent.com/85767/107262718-2ddfeb00-6a41-11eb-81aa-8884484e5010.png)</details> It looks like PRM has only 2 settings (play on host/do not play) while it needs 3. **Expected behavior** The sound shouldn't stop playing on remote machine, just like it doesn't do on native RD when above option is set. **Desktop (please complete the following information):** - OS: Win10 10.0.19042 - PRemoteM Version 0.5.8.2101192131
Author
Owner

@VShawn commented on GitHub (Feb 18, 2021):

image

this might fix only if we set line 158 to "1", no new options needs to add.


Is this ambiguous in your opinion? text will get cut off if I wrote whole statement "Play on remote computer"
image

<!-- gh-comment-id:781260677 --> @VShawn commented on GitHub (Feb 18, 2021): ![image](https://user-images.githubusercontent.com/10143738/108346581-802bc380-721a-11eb-88df-5d6fe21a9227.png) this might fix only if we set line 158 to "1", no new options needs to add. --- Is this ambiguous in your opinion? text will get cut off if I wrote whole statement "Play on remote computer" ![image](https://user-images.githubusercontent.com/10143738/108348820-37293e80-721d-11eb-93a4-8502fe1d98bb.png)
Author
Owner

@majkinetor commented on GitHub (Feb 18, 2021):

Is this ambiguous in your opinion? text will get cut off if I wrote whole statement "Play on remote computer"

Why not using single combobox and add 3 options ? That will solve your current interface and translation problems

Sound: [ disable, local, remote ]

unless its possible to play both local and remote (I 99% sure it isn't)

this might fix only if we set line 158 to "1", no new options needs to add.

That solution is semantically incorrect - you can't enable sound in branch that should be run if sound is disabled :) Go with the combo IMO. Its equiv to RDP native radio group but its way easier and non problematic in visual aspects.

FYI @VShawn

You can also link to the file directly instead of using screenshot, like this:

github.com/VShawn/PRemoteM@1cfa90ec43/PRM.Core/Protocol/RDP/Host/AxMsRdpClient09Host.xaml.cs (L143-L159)

<!-- gh-comment-id:781382224 --> @majkinetor commented on GitHub (Feb 18, 2021): > Is this ambiguous in your opinion? text will get cut off if I wrote whole statement "Play on remote computer" Why not using single combobox and add 3 options ? That will solve your current interface and translation problems ``` Sound: [ disable, local, remote ] ``` unless its possible to play both local and remote (I 99% sure it isn't) > this might fix only if we set line 158 to "1", no new options needs to add. That solution is semantically incorrect - you can't enable sound in branch that should be run if sound is disabled :) Go with the combo IMO. Its equiv to RDP native radio group but its way easier and non problematic in visual aspects. <details><summary>FYI @VShawn </summary> You can also link to the file directly instead of using screenshot, like this: https://github.com/VShawn/PRemoteM/blob/1cfa90ec43b5a58076490157c5f82e806ba902eb/PRM.Core/Protocol/RDP/Host/AxMsRdpClient09Host.xaml.cs#L143-L159 </details>
Author
Owner

@VShawn commented on GitHub (Feb 18, 2021):

Go with the combo IMO

ok... do it latter

You can also link to the file directly instead of using screenshot, like this

I knew, just lazy to find the file on GitHub 😓

<!-- gh-comment-id:781393725 --> @VShawn commented on GitHub (Feb 18, 2021): > Go with the combo IMO ok... do it latter > You can also link to the file directly instead of using screenshot, like this I knew, just lazy to find the file on GitHub :sweat:
Author
Owner

@majkinetor commented on GitHub (Feb 18, 2021):

I knew, just lazy to find the file on GitHub 😓

I supposed that, and its OK. Perhaps you missed that Go to file lets you do that by typing couple of letters... in this case axh :)

<!-- gh-comment-id:781410712 --> @majkinetor commented on GitHub (Feb 18, 2021): > I knew, just lazy to find the file on GitHub 😓 I supposed that, and its OK. Perhaps you missed that [Go to file](https://github.com/VShawn/PRemoteM/find/dev) lets you do that by typing couple of letters... in this case `axh` :)
Author
Owner

@VShawn commented on GitHub (Feb 19, 2021):

it works

image

Perhaps you missed that Go to file lets you do that by typing couple of letters

thanks for the tip!

<!-- gh-comment-id:781960400 --> @VShawn commented on GitHub (Feb 19, 2021): it works ![image](https://user-images.githubusercontent.com/10143738/108486989-b2e8c100-72d9-11eb-873f-92afb0883674.png) > Perhaps you missed that Go to file lets you do that by typing couple of letters thanks for the tip!
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/1Remote#2946
No description provided.