Bug 249439

Summary: When in the memo field of the transaction form, you can tab out in most accounts, but not investment accounts.
Product: [Applications] kmymoney Reporter: Jack <ostroffjh>
Component: generalAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal CC: agander93
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Jack 2010-08-29 19:26:43 UTC 
Version:           unspecified (using KDE 4.4.5) 
OS:                Linux

When entering or editing a transaction in the transaction form, in most accounts, you can tab through all the fields, and hitting tab in the last field goes back to the first.

Specifically in investment accounts, hitting tab in the memo field just enters a tab, it does not go on to the next field.

Reproducible: Didn't try



Expected Results:  
I assume this behavior should be consistent for all account types, and I prefer that tab move out of the memo field, as it is unlikely to really need to enter a tab character in this field.
Comment 1 Jack 2010-08-29 19:27:06 UTC 
KMyMoney 4.5.0
Comment 2 allan 2010-08-29 19:51:13 UTC 
(In reply to comment #1)
> KMyMoney 4.5.0

Hmm, but shift tab exits.  Without the transaction form, it seems to work as expected.
Comment 3 Jack 2010-08-29 20:37:07 UTC 
Shift-tab works correctly in the transaction form - backing up to the previous field.  It's only tab that inserts instead of going to the next field.
Comment 4 Alvaro Soliverez 2010-08-29 22:14:24 UTC 
It's weird, because the code is similar in both cases, but it doesn't work on investments.

Here's the investment code:

  KTextEdit* memo = new KTextEdit;
  memo->setTabChangesFocus(true);
  m_editWidgets["memo"] = memo;

This should work, though it doesn't.
Comment 5 Cristian Oneț 2010-09-05 19:14:21 UTC 
SVN commit 1171909 by conet:

BUG: 249439
Don't add the total edit widget in the tab order chain since it's just a QLabel (not a real edit widget) so it was breaking the tab order chain.

 M  +0 -3      transaction.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1171909
Comment 6 Cristian Oneț 2010-09-05 19:15:22 UTC 
SVN commit 1171910 by conet:

BUG: 249439
Don't add the total edit widget in the tab order chain since it's just a QLabel (not a real edit widget) so it was breaking the tab order chain.
Backport to the stable branch.

 M  +0 -3      transaction.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1171910