Skip to content

[v16] Ensure sharing remains enabled if the initial desktop connection attempt fails#55455

Merged
gzdunek merged 2 commits intobranch/v16from
gzdunek/backport-55350/v16
Jun 6, 2025
Merged

[v16] Ensure sharing remains enabled if the initial desktop connection attempt fails#55455
gzdunek merged 2 commits intobranch/v16from
gzdunek/backport-55350/v16

Conversation

@gzdunek
Copy link
Copy Markdown
Contributor

@gzdunek gzdunek commented Jun 5, 2025

Backport #55350 to branch/v16

EDIT: this problem doesn't happen on v16 as we refresh the page on Reconnect click, so we always start from the clear state.
But it's still worth to backport it.

…mpt fails (#55350)

* Ensure sharing remains enabled if the initial desktop connection attempt fails

* Refactor sharing state

* Add test

* Center "more actions" button vertically

(cherry picked from commit 1562144)
@gzdunek gzdunek force-pushed the gzdunek/backport-55350/v16 branch from b5cab1e to 1106394 Compare June 5, 2025 09:02
@gzdunek gzdunek added the no-changelog Indicates that a PR does not require a changelog entry label Jun 5, 2025
Copy link
Copy Markdown
Member

@ravicious ravicious left a comment

Choose a reason for hiding this comment

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

I verified that I'm able to connect to a desktop through a browser on v16.

@gzdunek gzdunek added this pull request to the merge queue Jun 6, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 6, 2025
@gzdunek gzdunek added this pull request to the merge queue Jun 6, 2025
Merged via the queue into branch/v16 with commit 2555799 Jun 6, 2025
41 checks passed
@gzdunek gzdunek deleted the gzdunek/backport-55350/v16 branch June 6, 2025 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/md ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants