Despite the fix for #267760, pressing Esc to close the search box still gives the "'Esc' is an ambiguous shortcut" message in some situations. I tracked it down: it happens if I open the Watched Nicks window while the search box is open. Steps to reproduce: - Open the search box (Ctrl-F). - Open the Watched Nicks window (for example, by pressing F4, usually accidentally while intending to hit F3). The search input field doesn't lose focus. - Press Esc. An "ambiguous shortcut" popup appears.
Only reproduceable if [ ] Focus new tabs is unchecked. otherwise the search box looses the focus on F4 press
Git commit 3e9bde736224f6646c070a5a8cd5ae5b0274f37f by eli mackenzie. Committed on 06/02/2013 at 02:12. Pushed by argonel into branch 'master'. Squelch spurious "Ambiguous Shortcut" warning SearchBar manages the shortcut for ESC, but does it in a way that ViewContainer can't account for. If Focus New Tabs is not enabled, the ViewContainer will reset the action's status without informing SearchBar. Now ViewContainer will send an event through the event-filter managing the shortcut so that it can deactivate the aciton, avoiding the error popup. M +21 -4 src/viewer/searchbar.cpp M +8 -1 src/viewer/viewcontainer.cpp http://commits.kde.org/konversation/3e9bde736224f6646c070a5a8cd5ae5b0274f37f