[PR #6573] [MERGED] fix: Keyboard shortcuts modal: Rearrange and add date shortcut "E" #60265

Closed
opened 2026-05-07 02:16:32 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6573
Author: @Juulz
Created: 1/6/2026
Status: ✅ Merged
Merged: 1/7/2026
Merged by: @youngcw

Base: master ← Head: keyboard-shortcuts-modal-update


📝 Commits (5)

  • b726310 Refactor keyboard shortcuts in KeyboardShortcutModal
  • 55a6b30 Update keyboard shortcuts modal text from 'General' to 'Global'
  • 7ae6fe2 Update VRT screenshots
  • f14c9b9 Add release notes for bugfix in KeybordShortcutModal
  • 9b8c527 Merge branch 'master' into keyboard-shortcuts-modal-update

📊 Changes

9 files changed (+130 additions, -93 deletions)

View changed files

📝 packages/desktop-client/e2e/help-menu.test.ts (+1 -1)
📝 packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-1-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-2-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-3-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-7-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-8-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-9-chromium-linux.png (+0 -0)
📝 packages/desktop-client/src/components/modals/KeyboardShortcutModal.tsx (+123 -92)
➕ upcoming-release-notes/6573.md (+6 -0)

📄 Description

Fixed KeyBoardShortcutModal.tsx:

Changed "General" to "Global" due to test fail. Changed test from "General" to "Global".
Added sections to break apart Account shortcuts. 27 is too many to find what you are looking for.
Added one contextual shortcuts to two sections.. "F" as "Open filter dropdown to "Account page general" and as "Filter to the selected transactions" to "Account transaction management"
"T" is "Add transaction" to "Account page general"
"E" is "Set date for selected transactions" to "Account transaction management"
Rearranged shortcuts to match the order in PR https://github.com/actualbudget/actual/pull/6417
Found "T" description mismatch when working on Tips & Tricks doc. Fixed in https://github.com/actualbudget/actual/pull/6416


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/6573 **Author:** [@Juulz](https://github.com/Juulz) **Created:** 1/6/2026 **Status:** ✅ Merged **Merged:** 1/7/2026 **Merged by:** [@youngcw](https://github.com/youngcw) **Base:** `master` ← **Head:** `keyboard-shortcuts-modal-update` --- ### 📝 Commits (5) - [`b726310`](https://github.com/actualbudget/actual/commit/b726310fc9fec9afc44b66bae70e9618d3d76f56) Refactor keyboard shortcuts in KeyboardShortcutModal - [`55a6b30`](https://github.com/actualbudget/actual/commit/55a6b308d83fc9752c5db701a03e942d6619e0a5) Update keyboard shortcuts modal text from 'General' to 'Global' - [`7ae6fe2`](https://github.com/actualbudget/actual/commit/7ae6fe2055836d469d84ccc69dc0a657af668a3e) Update VRT screenshots - [`f14c9b9`](https://github.com/actualbudget/actual/commit/f14c9b94bb216baaf5c0eeea6c552a59485adeb2) Add release notes for bugfix in KeybordShortcutModal - [`9b8c527`](https://github.com/actualbudget/actual/commit/9b8c527a1139c7b1520b9ed7d82c6ddb572fdb06) Merge branch 'master' into keyboard-shortcuts-modal-update ### 📊 Changes **9 files changed** (+130 additions, -93 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/e2e/help-menu.test.ts` (+1 -1) 📝 `packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-1-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-2-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-3-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-7-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-8-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/help-menu.test.ts-snapshots/Help-menu-Check-the-keyboard-shortcuts-modal-visuals-9-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/src/components/modals/KeyboardShortcutModal.tsx` (+123 -92) ➕ `upcoming-release-notes/6573.md` (+6 -0) </details> ### 📄 Description Fixed KeyBoardShortcutModal.tsx: Changed "General" to "Global" due to test fail. Changed test from "General" to "Global". Added sections to break apart Account shortcuts. 27 is too many to find what you are looking for. Added one contextual shortcuts to two sections.. "F" as "Open filter dropdown to "Account page general" and as "Filter to the selected transactions" to "Account transaction management" "T" is "Add transaction" to "Account page general" "E" is "Set date for selected transactions" to "Account transaction management" Rearranged shortcuts to match the order in PR https://github.com/actualbudget/actual/pull/6417 Found "T" description mismatch when working on Tips & Tricks doc. Fixed in https://github.com/actualbudget/actual/pull/6416 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-07 02:16:32 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#60265