Skip to content

Conversation

@jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Oct 31, 2025

Additional details

We recently added this keyboard shortcut, but it's not in the keyboard shortcut dialog. https://github.com/cypress-io/cypress-services/pull/12000

Steps to test

  • yarn cypress:open
  • Click on keyboard shortcuts in the bottom left of the test runner

How has the user experience changed?

Before

Screenshot 2025-10-31 at 11 20 50 AM

After (Mac)

Screenshot 2025-10-31 at 11 36 21 AM

After (Windows/Linux)

Screenshot 2025-10-31 at 11 36 30 AM

PR Tasks


Note

Displays the Studio save shortcut in the Keyboard Shortcuts modal with OS-specific keys (⌘+s or Ctrl+s), adds tests, and updates i18n and changelog.

  • UI (Keyboard Shortcuts Modal):
    • Show platform-specific Studio save shortcut + s (macOS) or Ctrl + s (Windows/Linux) in packages/app/src/navigation/KeyboardBindingsModal.vue.
    • Refactor key rendering to handle + separators and spacing; compute keyBindings via runner platform.
  • Tests:
    • Add comprehensive tests in packages/app/src/navigation/KeyboardBindingsModal.cy.tsx for rendering, platform-specific keys, fallback behavior, and modal close events.
  • i18n:
    • Add sidebar.keyboardShortcuts.studioSave to packages/frontend-shared/src/locales/en-US.json.
  • Changelog:
    • Note the new shortcut display in cli/CHANGELOG.md.

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

@jennifer-shehane jennifer-shehane self-assigned this Oct 31, 2025
@cypress
Copy link

cypress bot commented Oct 31, 2025

cypress    Run #67093

Run Properties:  status check passed Passed #67093  •  git commit ea7d825353: Merge branch 'develop' into keyboard-dialog-save-studio
Project cypress
Branch Review keyboard-dialog-save-studio
Run status status check passed Passed #67093
Run duration 16m 55s
Commit git commit ea7d825353: Merge branch 'develop' into keyboard-dialog-save-studio
Committer Jennifer Shehane
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 3
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.

Studio: Add new keyboard shortcut to keyboard shortcuts dialog in App

2 participants