Bug 338296

Summary:	In Ledger: Keyboard shortcut for Search (Alt-E) does not work
Product:	[Applications] kmymoney	Reporter:	BobSCA <bobs_spam_0>
Component:	general	Assignee:	Christian David <christian-david>
Status:	RESOLVED FIXED
Severity:	minor	CC:	christian-david, kmymoney-devel, onet.cristian
Priority:	NOR
Version:	4.6.4
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	http://commits.kde.org/kmymoney/fc539ad85c99c78dce648020b819d46c534c41af	Version Fixed In:	4.7.2
Sentry Crash Report:

Description BobSCA 2014-08-15 14:29:35 UTC

Alt-E does not select the ledger search box as it instead is used for the Edit menu. Search shortcut should be changed to to Alt-R.

Reproducible: Always

Steps to Reproduce:
1.Go to ledger
2.Hit Alt-E as suggested by the underlined "e" in the word Search

Actual Results:  
The Edit menu drops down

Expected Results:  
The keyboard focus should go to the Search box.

Comment 1 Christian David 2015-01-03 13:04:54 UTC

Added patch: https://git.reviewboard.kde.org/r/121803/

Comment 2 Christian David 2015-01-04 16:40:47 UTC

Git commit bf29b86bcb28c9a1837f351a0cb83f791c779eda by Christian Dávid.
Committed on 03/01/2015 at 12:42.
Pushed by christiand into branch 'master'.

Changed label for serach bar in ledger from "Search" to "Filter"

The field is actually more filter than search.

This also fixes the bug "In Ledger: Keyboard shortcut for Search (Alt-E)
does not work".

I could not change the label in scheduled transactions view because it
is hard coded in the library. But this should be done as well.
FIXED-IN: 4.7.2
REVIEW: 121803

M  +1    -1    kmymoney/widgets/registersearchline.cpp

http://commits.kde.org/kmymoney/bf29b86bcb28c9a1837f351a0cb83f791c779eda

Comment 3 Christian David 2015-01-04 17:22:55 UTC

Git commit fc539ad85c99c78dce648020b819d46c534c41af by Christian Dávid.
Committed on 03/01/2015 at 12:42.
Pushed by christiand into branch '4.7'.

Changed label for serach bar in ledger from "Search" to "Filter"

The field is actually more filter than search.

This also fixes the bug "In Ledger: Keyboard shortcut for Search (Alt-E)
does not work".

I could not change the label in scheduled transactions view because it
is hard coded in the library. But this should be done as well.
FIXED-IN: 4.7.2
REVIEW: 121803
(cherry picked from commit bf29b86bcb28c9a1837f351a0cb83f791c779eda)

M  +1    -1    kmymoney/widgets/registersearchline.cpp

http://commits.kde.org/kmymoney/fc539ad85c99c78dce648020b819d46c534c41af