Bug 249439 - When in the memo field of the transaction form, you can tab out in most accounts, but not investment accounts.
Summary: When in the memo field of the transaction form, you can tab out in most accou...
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-29 19:26 UTC by Jack
Modified: 2010-09-05 19:15 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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