Allow using Tab key for bindings

Tab was always consumed by focus cycling even though that should be at the lowest priority.

IssueID #244

2 files changed, 11 insertions, 10 deletions

diff --git a/src/app.c b/src/app.c
index 2ccadca8..d8053315 100644
--- a/src/app.c
+++ b/src/app.c
@@ -1023,6 +1023,16 @@ void processEvents_App(enum iAppEventMode eventMode) {
                     /* There may be a key bindings for this. */
                     wasUsed = processEvent_Keys(&ev);
                 }
+                if (!wasUsed) {
+                    /* Focus cycling. */
+                    if (ev.type == SDL_KEYDOWN && ev.key.keysym.sym == SDLK_TAB) {
+                        setFocus_Widget(findFocusable_Widget(focus_Widget(),
+                                                             ev.key.keysym.mod & KMOD_SHIFT
+                                                                 ? backward_WidgetFocusDir
+                                                                 : forward_WidgetFocusDir));
+                        wasUsed = iTrue;
+                    }
+                }
                 if (ev.type == SDL_USEREVENT && ev.user.code == command_UserEventCode) {
 #if defined (iPlatformAppleDesktop)
                     handleCommand_MacOS(command_UserEvent(&ev));
diff --git a/src/ui/widget.c b/src/ui/widget.c
index 1ef2360c..3eea761e 100644
--- a/src/ui/widget.c
+++ b/src/ui/widget.c
@@ -906,16 +906,7 @@ static iBool scrollOverflow_Widget_(iWidget *d, int delta) {
 }
 
 iBool processEvent_Widget(iWidget *d, const SDL_Event *ev) {
-    if (ev->type == SDL_KEYDOWN) {
-        if (ev->key.keysym.sym == SDLK_TAB) {
-            setFocus_Widget(findFocusable_Widget(focus_Widget(),
-                                                 ev->key.keysym.mod & KMOD_SHIFT
-                                                     ? backward_WidgetFocusDir
-                                                     : forward_WidgetFocusDir));
-            return iTrue;
-        }
-    }
-    else if (d->flags & commandOnClick_WidgetFlag &&
+    if (d->flags & commandOnClick_WidgetFlag &&
              (ev->type == SDL_MOUSEBUTTONDOWN || ev->type == SDL_MOUSEBUTTONUP) &&
              (mouseGrab_Widget() == d || contains_Widget(d, init_I2(ev->button.x, ev->button.y)))) {
         postCommand_Widget(d,