Skip to content

feat(applications): add shorcut for application modal#10

Open
DonnaIsabel97 wants to merge 1 commit into
Joun-Mikhail:mainfrom
DonnaIsabel97:fix-issue-7
Open

feat(applications): add shorcut for application modal#10
DonnaIsabel97 wants to merge 1 commit into
Joun-Mikhail:mainfrom
DonnaIsabel97:fix-issue-7

Conversation

@DonnaIsabel97

Copy link
Copy Markdown

Summary

Adds the keyboard shortcut requested in #7. On the Applications page, pressing n now opens the Add application modal when the user is not typing in a form field.

Changes

  • Added a guarded keydown listener on the Applications page.
  • Pressing n opens the Add application modal.
  • Prevented the n key from being inserted into the Role title field when the modal opens.
  • Ignored the shortcut when focus is inside an input, textarea, select, or editable element.
  • Preserved existing modal behavior, including closing with Esc.

Checklist

  • Tests added or updated
  • ruff, mypy, and pytest pass for the backend
  • npm run lint, npm run typecheck, npm test, and npm run build pass for the frontend (if touched)
  • Documentation updated where relevant
  • No secrets, debug output, or commented-out code left behind

@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown

@DonnaIsabel97 is attempting to deploy a commit to the joun-mikhail's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant