Bug 504316 - Start editing an existing split when pressing Return
Summary: Start editing an existing split when pressing Return
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: ux-ui (other bugs)
Version First Reported In: 5.1.3
Platform: openSUSE Linux
: NOR wishlist
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-05-16 08:15 UTC by Ingo Klöcker
Modified: 2025-05-17 07:02 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed/Implemented In: 5.2
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ingo Klöcker 2025-05-16 08:15:40 UTC
It would be very convenient if I could edit an existing split the same way that I can edit an existing transaction in the ledger, i.e. by pressing Return when the transaction is selected. Currently, I have to grab the mouse to edit a split (or press the Menu button that my keyboard luckily still has).

STEPS TO REPRODUCE
1. Have a transaction with splits.
2. Edit the transaction.
3. Open the split editor.
4. Try to edit the first split without using the mouse.

OBSERVED RESULT
If you are lucky then your keyboard has a Menu button which you can press to get the context menu with the Edit entry. Maybe there's a hidden keyboard shortcut I don't know about.
Pressing Return doesn't start editing the selected split but seems to trigger the OK button (although the Cancel button is clearly highlighted as default button by Qt)

EXPECTED RESULT
I can press Return to start editing a split in the split window just like I can press Return to start editing a transaction in the ledger.
Comment 1 Thomas Baumgart 2025-05-17 07:02:00 UTC
Git commit cb0081d97279267d15dbf9c74745fbf568f7247f by Thomas Baumgart.
Committed on 17/05/2025 at 07:01.
Pushed by tbaumgart into branch 'master'.

Improve keyboard handling of split editor

While editing a split, closing the editor using the ESC
key also closed the whole split dialog. This change
prevents to close the dialog in this case so that the
user can continue to edit other splits and sets
the initial focus on the split view.

M  +3    -1    kmymoney/views/newspliteditor.cpp
M  +2    -0    kmymoney/views/splitdialog.cpp

https://invent.kde.org/office/kmymoney/-/commit/cb0081d97279267d15dbf9c74745fbf568f7247f
Comment 2 Thomas Baumgart 2025-05-17 07:02:02 UTC
Git commit fbe0d432380a26bd813386d45e7326984400bbd0 by Thomas Baumgart.
Committed on 17/05/2025 at 07:01.
Pushed by tbaumgart into branch 'master'.

Allow Enter key to start editing split
FIXED-IN: 5.2

M  +35   -0    kmymoney/views/splitview.cpp
M  +1    -0    kmymoney/views/splitview.h

https://invent.kde.org/office/kmymoney/-/commit/fbe0d432380a26bd813386d45e7326984400bbd0