Skip to content

Commit 9c2f94b

Browse files
authored
fix(site): remove erroneous "install Cursor" notification for Cursor Desktop (#20875)
### Summary This change removes the erroneous “Install Cursor” notification in the desktop environment when Cursor is already installed. The issue is the timeout of 500ms was simply too short for Cursor Desktop to respond in time, so I increased it to 1500ms. This seems to consistently work, but it could easily be increased to 2000ms to be safe. ### Issue Fixes #20289 ### Testing - Verified that when Cursor is installed, the notification no longer appears ### QA #### Before https://github.com/user-attachments/assets/facd2e74-6eb7-47ac-935d-7b11974648a0 #### After https://github.com/user-attachments/assets/ced817d8-ca0c-428c-8436-5f30ffc6134b
1 parent 6238a99 commit 9c2f94b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

site/src/modules/apps/useAppLink.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ export const useAppLink = (
5555
// When browser recognizes the protocol and is able to navigate to the app,
5656
// it will blur away, and will stop the timer. Otherwise,
5757
// an error message will be displayed.
58-
const openAppExternallyFailedTimeout = 500;
58+
const openAppExternallyFailedTimeout = 1500;
5959
const openAppExternallyFailed = setTimeout(() => {
6060
// Check if this is a JetBrains IDE app
6161
// starts with "jetbrains-gateway://connect#type=coder" (from https://registry.coder.com/modules/coder/jetbrains-gateway)

0 commit comments

Comments
 (0)