1
0
mirror of https://github.com/cookiengineer/audacity synced 2025-06-23 15:50:05 +02:00

Fix toolbar grabbers being in tab traversal

In commit 89e33da, the override of AcceptsFocus() was removed from the Grabber class so that ESC could cancel the dragging of a toolbar. However this meant that the toolbar grabbers were included in the tab traversal of the toolbars.

The fix is to override AcceptsFocusFromKeyboard, rather than AcceptsFocus().
This commit is contained in:
David Bailes 2016-06-07 09:27:02 +01:00
parent a8f4a94983
commit 78d7ce8aa7

View File

@ -106,8 +106,9 @@ class Grabber final : public wxWindow
// not a need to dock/float a toolbar from the keyboard. If this
// changes, remove this and add the necessary keyboard movement
// handling.
// PRL: Commented out so the ESC key can stop dragging.
// bool AcceptsFocus() const {return false;}
// Note that AcceptsFocusFromKeyboard() rather than AcceptsFocus()
// is overridden so that ESC can cancel toolbar drag.
bool AcceptsFocusFromKeyboard() const override {return false;}
void PushButton(bool state);