Bug 338296 - In Ledger: Keyboard shortcut for Search (Alt-E) does not work
Summary: In Ledger: Keyboard shortcut for Search (Alt-E) does not work
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: 4.6.4
Platform: Microsoft Windows Microsoft Windows
: NOR minor
Target Milestone: ---
Assignee: Christian David
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-15 14:29 UTC by BobSCA
Modified: 2015-01-04 17:22 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.7.2
Sentry Crash Report:


Attachments

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