Bug 504316

Summary: Start editing an existing split when pressing Return
Product: [Applications] kmymoney Reporter: Ingo Klöcker <kloecker>
Component: ux-uiAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version First Reported In: 5.1.3   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: https://invent.kde.org/office/kmymoney/-/commit/fbe0d432380a26bd813386d45e7326984400bbd0 Version Fixed In: 5.2
Sentry Crash Report:

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