Skip to content

Conversation

@astone123
Copy link
Contributor

@astone123 astone123 commented Nov 6, 2025

  • Closes N/A

Additional details

This PR adds a logo for the Cursor editor on the external editor configuration dropdown. Before we were just showing the generic terminal icon

Steps to test

  • Open Cypress
  • Go to settings
  • Expand Device Settings
  • Click the external editor dropdown
  • Verify Cursor logo appears next to its name

How has the user experience changed?

Before
Screenshot 2025-11-06 at 5 45 28 PM

After
Screenshot 2025-11-06 at 5 45 01 PM

PR Tasks


Note

Adds a Cursor logo and config to display it in the external editor dropdown via a new logos icon collection.

  • UI (external editor selector):
    • Import and map cursor icon in src/gql-components/ChooseExternalEditor.vue.
  • Assets:
    • Add src/assets/logos/cursor.svg.
  • Build/Icons:
    • Configure vite.config.mjs with custom logos icon collection pointing to src/assets/logos.
  • Types:
    • Update vue-shims.d.ts to declare ~icons/logos/* and align FC typings.

Written by Cursor Bugbot for commit 9ed122d. This will update automatically on new commits. Configure here.

@astone123 astone123 self-assigned this Nov 6, 2025
@cypress
Copy link

cypress bot commented Nov 6, 2025

cypress    Run #67206

Run Properties:  status check passed Passed #67206  •  git commit 9ed122dcd7: Merge branch 'develop' into cursor-external-logo
Project cypress
Branch Review cursor-external-logo
Run status status check passed Passed #67206
Run duration 17m 11s
Commit git commit 9ed122dcd7: Merge branch 'develop' into cursor-external-logo
Committer Adam Stone-Lord
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 4
Tests that did not run due to a developer annotating a test with .skip  Pending 131
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 5535
View all changes introduced in this branch ↗︎

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.

2 participants