Skip to content

feat: screen picker for windows desktop screen sharing#1124

Merged
Dadadah merged 3 commits intostoatchat:mainfrom
Dadadah:feat/screenpicker
Apr 30, 2026
Merged

feat: screen picker for windows desktop screen sharing#1124
Dadadah merged 3 commits intostoatchat:mainfrom
Dadadah:feat/screenpicker

Conversation

@Dadadah
Copy link
Copy Markdown
Member

@Dadadah Dadadah commented Apr 18, 2026

This PR adds a screen picker modal that shows when a user initiates a screen share on Windows or Linux X11 using the desktop app. All other platforms, including Linux Wayland, do not use the screen picker modal.

Closes #1068

Requires stoatchat/for-desktop#207 (Not required for merge, but will not work until PR is merged and deployed to desktop)

Blocks #1070

Requires #1111

How was this PR tested?

Screen share was initiated using a custom desktop build on Windows. Screen share was also tested on Linux Wayland and still works without the picker.

Screenshots & Screencasts

screencast.mp4

(This video is a bit choppy, that's due to the encoding. I had to make it smaller to upload it to github)

Checklist:

  • I have carefully read the contributing guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR
  • I have confirmed that any new dependencies are strictly necessary
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code

Please declare, if any, LLM usage involved in creating this PR

Generative AI was not used in the writing of this PR.

@Dadadah Dadadah force-pushed the feat/screenpicker branch from 748b46c to 13419a6 Compare April 19, 2026 17:39
@Dadadah Dadadah marked this pull request as ready for review April 19, 2026 18:25
@Dadadah Dadadah requested review from a team as code owners April 19, 2026 18:25
@gabrielfordevelopment
Copy link
Copy Markdown
Contributor

image Maybe we could make this button red since this is a destructive button - to turn OFF the share

@gabrielfordevelopment gabrielfordevelopment added ux review This needs UX review enhancement New feature or request labels Apr 22, 2026
Comment thread packages/client/components/rtc/state.tsx
Comment thread packages/client/components/modal/modals/ScreenSharePicker.tsx
Copy link
Copy Markdown
Contributor

@gabrielfordevelopment gabrielfordevelopment left a comment

Choose a reason for hiding this comment

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

Sweet!

Copy link
Copy Markdown
Contributor

@infi infi left a comment

Choose a reason for hiding this comment

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

LGTM as per Stoat conversation

@Dadadah Dadadah merged commit 7c1f1e6 into stoatchat:main Apr 30, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ux review This needs UX review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Windows screen picker for Desktop

4 participants