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:	general	Assignee:	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:

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