Skip to content

docs: reorganize remote desktop docs #18716

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Conversation

matifali
Copy link
Member

@matifali matifali commented Jul 2, 2025

  • Reorganize each option in two sections: Web and Desktop Client
  • Moves the warning about UDP connections to the bottom
  • Move Coder Desktop as the first option
  • Links the Coder Desktop RDP module

Preview: https://coder.com/docs/@remote-desktop-module/user-guides/workspace-access/remote-desktops

@matifali matifali marked this pull request as draft July 2, 2025 14:15
@matifali matifali marked this pull request as ready for review July 3, 2025 03:42
phorcys420 pushed a commit to coder/registry that referenced this pull request Jul 3, 2025
## Description

Use more descriptive names

Depends on coder/coder#coder/coder#18716

---

## Type of Change

- [ ] New module
- [ ] Bug fix
- [ ] Feature/enhancement
- [x] Documentation
- [ ] Other

## Troubleshooting

### RDP UDP Issues
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I agree with putting this at the end instead of in the RDP section. Seems less likely to get read.

Copy link
Member Author

@matifali matifali Jul 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a rendering issue with Notes in Tabs, so I moved it to the end. Looks like we are fixing it soon: https://github.com/coder/coder.com/pull/265

I can move it back once that PR gets merged.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved the Note back up.


The common way to use remote desktops with Coder is through VNC.
The common way to connect to a Windows workspace is using Remote Desktop Protocol (RDP). RDP is a proprietary protocol developed by Microsoft that allows users to remotely control a computer over a network connection. It provides a graphical user interface (GUI) for remote access, enabling users to interact with the remote system as if they were sitting in front of it.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This reads as patronizing to me. Remote desktop implies a GUI, and given that we don't provide instructions on how to enable RDP in the workspace, I think we should assume the reader has some idea what RDP is.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to remove the sentence talking about what RDP is.

@matifali matifali requested a review from bjornrobertsson July 3, 2025 11:22
@matifali matifali requested a review from bpmct July 3, 2025 20:40
> New-ItemProperty -Path 'HKLM:\SOFTWARE\Policies\Microsoft\Windows NT\Terminal Services' -Name "SelectTransport" -Value 1 -PropertyType DWORD -Force
> Restart-Service -Name "TermService" -Force
> ```
You can use the [RDP Desktop](https://registry.coder.com/modules/coder/local-windows-rdp) module to add a single-click button to open an RDP session in the browser.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest adding a screenshot or GIF here of what it looks like from the desktop client. Would significantly improve the visual identity of the first part of the page and show what the Desktop UX looks like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants