[PR #6614] [CLOSED] desktop-client: Add useKey = true to every useHotkeys usage #41062

Closed
opened 2026-04-23 13:52:04 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6614
Author: @hw0lff
Created: 1/10/2026
Status: Closed

Base: masterHead: usekey-in-usehotkeys


📝 Commits (1)

  • 492bb0a desktop-client: Add useKey = true to every useHotkeys usage

📊 Changes

7 files changed (+14 additions, -0 deletions)

View changed files

📝 packages/desktop-client/src/components/Titlebar.tsx (+2 -0)
📝 packages/desktop-client/src/components/accounts/Header.tsx (+4 -0)
📝 packages/desktop-client/src/components/budget/DynamicBudgetTable.tsx (+3 -0)
📝 packages/desktop-client/src/components/filters/FiltersMenu.tsx (+1 -0)
📝 packages/desktop-client/src/components/reports/Overview.tsx (+1 -0)
📝 packages/desktop-client/src/components/transactions/SelectedTransactionsButton.tsx (+2 -0)
📝 packages/desktop-client/src/components/transactions/TransactionsTable.tsx (+1 -0)

📄 Description

I pressed the wrong button too early to submit this PR.

This is supposed to test if I can fix the keyboard layout issue described in https://github.com/actualbudget/actual/issues/6278

However, I did not fully test this yet and ran out of time for this today.
So I just wanted submit this PR to not forget about it.

I will add the release notes if this actually fixes the issue and is desired to be merged.


🔄 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/6614 **Author:** [@hw0lff](https://github.com/hw0lff) **Created:** 1/10/2026 **Status:** ❌ Closed **Base:** `master` ← **Head:** `usekey-in-usehotkeys` --- ### 📝 Commits (1) - [`492bb0a`](https://github.com/actualbudget/actual/commit/492bb0a6ad820f8f360854e47164a90791f88f60) desktop-client: Add useKey = true to every useHotkeys usage ### 📊 Changes **7 files changed** (+14 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/Titlebar.tsx` (+2 -0) 📝 `packages/desktop-client/src/components/accounts/Header.tsx` (+4 -0) 📝 `packages/desktop-client/src/components/budget/DynamicBudgetTable.tsx` (+3 -0) 📝 `packages/desktop-client/src/components/filters/FiltersMenu.tsx` (+1 -0) 📝 `packages/desktop-client/src/components/reports/Overview.tsx` (+1 -0) 📝 `packages/desktop-client/src/components/transactions/SelectedTransactionsButton.tsx` (+2 -0) 📝 `packages/desktop-client/src/components/transactions/TransactionsTable.tsx` (+1 -0) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes. Try running yarn generate:release-notes *before* pushing your PR for an interactive experience. --> I pressed the wrong button too early to submit this PR. This is supposed to test if I can fix the keyboard layout issue described in https://github.com/actualbudget/actual/issues/6278 However, I did not fully test this yet and ran out of time for this today. So I just wanted submit this PR to not forget about it. I will add the release notes if this actually fixes the issue and is desired to be merged. --- <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-04-23 13:52:04 -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#41062