lots of unrelated fixes

src/Ewmh.cc

@@ -662,15 +662,7 @@ void Ewmh::updateState(FluxboxWindow &win) {
 }
 
 void Ewmh::updateLayer(FluxboxWindow &win) {
-    //!! TODO _NET_WM_WINDOW_TYPE
-    /*
-    if (win.getLayer() == Fluxbox::instance()->getAboveDockLayer()) {
-        // _NET_WM_STATE_BELOW
-
-    } else if (win.getLayer() == Fluxbox::instance()->getBottomLayer()) {
-        // _NET_WM_STATE_ABOVE
-    }
-    */
+    updateState(win);
 }
 
 void Ewmh::updateHints(FluxboxWindow &win) {

src/FbTk/Menu.cc

@@ -965,6 +965,10 @@ void Menu::motionNotifyEvent(XMotionEvent &me) {
         if (w == m_active_index)
             return;
 
+        // if another menu is focused, change focus to this one, so arrow keys
+        // work as expected
+        if (s_focused != this && s_focused != 0)
+            grabInputFocus();
 
         if (validIndex(m_active_index) && w != m_active_index) {
             int old_active_index = m_active_index;

src/Keys.cc

@@ -339,7 +339,7 @@ bool Keys::doAction(int type, unsigned int mods, unsigned int key) {
     // grab "None Escape" to exit keychain in the middle
     unsigned int esc = FbTk::KeyUtil::getKey("Escape");
 
-    if (temp_key && temp_key->keylist.size()) { // emacs-style
+    if (temp_key && !temp_key->keylist.empty()) { // emacs-style
         if (!saved_keymode)
             saved_keymode = m_keylist;
         next_key = temp_key;

src/fluxbox.cc

@@ -733,7 +733,9 @@ void Fluxbox::handleEvent(XEvent * const e) {
         for (; it != it_end; ++it) {
             if ( (*it)->screenNumber() ==
                  FbTk::Menu::focused()->fbwindow().screenNumber()) {
-                FocusControl::revertFocus(**it);
+                FocusControl::setFocusedWindow(0);
+                m_revert_screen = *it;
+                m_revert_timer.start();
                 break; // found the screen, no more search
             }
         }