Skip to content
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

[Bug]: Cypress testing requires cmd on Mac and ctrl on Linux/Windows #125

Open
10 tasks
shapeseas opened this issue Feb 21, 2025 · 3 comments · May be fixed by #128
Open
10 tasks

[Bug]: Cypress testing requires cmd on Mac and ctrl on Linux/Windows #125

shapeseas opened this issue Feb 21, 2025 · 3 comments · May be fixed by #128
Assignees
Labels
bug Something isn't working

Comments

@shapeseas
Copy link
Collaborator

What happened?

Testing is difficult to switch between OSs because the code editor relies on cmd / ctrl to delete all text. Create a way to detect system OS to avoid this annoyance.

What browsers do you see the problem on?

  • Firefox
  • Chrome
  • Safari
  • Microsoft Edge

What operating system do you see the problem on?

  • Windows
  • Mac
  • Linux

How did we discover the bug?

  • Reported by participant
  • Observed on Sentry
  • Found in dev

Priority

Inconvenience for developers

@shapeseas shapeseas added the bug Something isn't working label Feb 21, 2025
@dankim444
Copy link
Contributor

@shapeseas I can take this on

@dankim444 dankim444 self-assigned this Feb 21, 2025
@dankim444
Copy link
Contributor

You can get the underlying os through Cypress.platform it seems through this https://docs.cypress.io/api/cypress-api/platform

@shapeseas
Copy link
Collaborator Author

Sounds good, thanks @dankim444 !

@dankim444 dankim444 linked a pull request Feb 27, 2025 that will close this issue
@dankim444 dankim444 linked a pull request Feb 27, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants