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

Add transaction panel keyboard shortcuts #490

Merged
merged 3 commits into from Oct 8, 2017

Conversation

Projects
None yet
2 participants
@tuxlifan
Contributor

tuxlifan commented Oct 8, 2017

Tested only on Linux.

In the "Transaction Info" dialog press:

  • Alt+t to put focus on the "Transfers:" table view.
  • Alt+s to Save and close the dialog.

tuxlifan added some commits Oct 8, 2017

Qt: Add Alt+T shortcut to select Transactions table in Transaction In…
…fo dialog

It would be nice to also add an underline to the "T" in the self.amount
QLabel as visual clue for the user.
@tuxlifan

This comment has been minimized.

Contributor

tuxlifan commented Oct 8, 2017

Why?

I recently added a bunch of split transactions via QIF import (crafted from the abomination that my bank is spitting out as "csv" and used to track 'hidden' fees in foreign currency payments) which had each one category/account target missing (because not deterministic without human decision making), so I ended up with a lot of manual corrections to do. (Fortunately I figured out at least that even though they were not shown anymore with the Unassigned filter since they had one part of the split assigned, in Transaction list I could sort by the To column and they would show up with "Unassigned" as first transfer "account" – but I digress...)

I got fed up with TABing to the transfers for the n'th+1 time and after filling in the missing transfer account having to use the mouse to confirm my change and get rid of the dialog, so came up with this.

Hope it is useful.

@hsoft

This comment has been minimized.

Owner

hsoft commented Oct 8, 2017

I agree about the irritation. For me, it's not so much the repetitive tabbing, but more the "needs to use the mouse to get out of the transfers table". I always was too lazy to do the fix :) Thanks for the PR, I'm merging.

I'm curious about your "unassigned" problem. The Unassigned filter is supposed to show all transactions that have at least one unassigned split. Did it not do that? If it didn't, you might have uncovered a bug.

@hsoft hsoft merged commit d92d00b into hsoft:master Oct 8, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@tuxlifan tuxlifan deleted the tuxlifan:transaction_panel_keyboard_shortcuts branch Mar 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment