fixed focus issue

1 files changed, 33 insertions, 131 deletions

diff --git a/src/fluxbox.cc b/src/fluxbox.cc
index 9cad953..439ed8f 100644
--- a/src/fluxbox.cc
+++ b/src/fluxbox.cc
@@ -22,7 +22,7 @@
 // FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
 // DEALINGS IN THE SOFTWARE.
 
-// $Id: fluxbox.cc,v 1.214 2003/12/21 22:42:31 fluxgen Exp $
+// $Id: fluxbox.cc,v 1.215 2003/12/30 20:56:40 fluxgen Exp $
 
 #include "fluxbox.hh"
 
@@ -194,13 +194,6 @@ setFromString(const char *strval) {
         setDefaultValue();
 }
 
-template<>
-void FbTk::Resource<long>::
-setFromString(const char *strval) {     
-    if (sscanf(strval, "%ld", &m_value) != 1)
-        setDefaultValue();
-}
-
 //-----------------------------------------------------------------
 //---- manipulators for int, bool, and some enums with Resource ---
 //-----------------------------------------------------------------
@@ -266,14 +259,6 @@ getString() {
 }
 
 template<>
-string FbTk::Resource<long>::
-getString() {
-    char tmpstr[128];
-    sprintf(tmpstr, "%ld", m_value);
-    return string(tmpstr);
-}
-
-template<>
 void FbTk::Resource<Fluxbox::Layer>::
 setFromString(const char *strval) {
     int tempnum = 0;
@@ -322,6 +307,20 @@ getString() {
         return string(tmpstr);
     }
 }
+template<>
+void FbTk::Resource<long>::
+setFromString(const char *strval) {   
+    if (sscanf(strval, "%ld", &m_value) != 1)
+        setDefaultValue();
+}
+ 
+
+string FbTk::Resource<long>::
+getString() {
+    char tmpstr[128];
+    sprintf(tmpstr, "%ld", m_value);
+    return string(tmpstr);
+}
 
 static Window last_bad_window = None;
 namespace {
@@ -393,11 +392,12 @@ Fluxbox::Fluxbox(int argc, char **argv, const char *dpy_name, const char *rcfile
       m_rc_cache_life(m_resourcemanager, 5, "session.cacheLife", "Session.CacheLife"),
       m_rc_cache_max(m_resourcemanager, 200, "session.cacheMax", "Session.CacheMax"),
       m_rc_auto_raise_delay(m_resourcemanager, 250, "session.autoRaiseDelay", "Session.AutoRaiseDelay"),
-      m_focused_window(0),
+      m_focused_window(0), m_masked_window(0),
       m_mousescreen(0),
       m_keyscreen(0),
       m_watching_screen(0), m_watch_keyrelease(0),
       m_last_time(0),
+      m_masked(0),
       m_rc_file(rcfilename ? rcfilename : ""),
       m_argv(argv), m_argc(argc),
       m_starting(true),
@@ -405,9 +405,7 @@ Fluxbox::Fluxbox(int argc, char **argv, const char *dpy_name, const char *rcfile
       m_server_grabs(0),
       m_randr_event_type(0),
       m_RC_PATH("fluxbox"),
-      m_RC_INIT_FILE("init"),
-      m_focus_revert_screen(0)
-{
+      m_RC_INIT_FILE("init") {
       
     
     if (s_singleton != 0)
@@ -552,7 +550,7 @@ Fluxbox::Fluxbox(int argc, char **argv, const char *dpy_name, const char *rcfile
             m_atomhandler[atomh]->initForScreen(*screen);
         }
 
-        revertFocus(*screen, false); // make sure focus style is correct
+        revertFocus(*screen); // make sure focus style is correct
 
     } // end init screens
 
@@ -624,10 +622,6 @@ void Fluxbox::eventLoop() {
             } else {
                 last_bad_window = None;
                 handleEvent(&e);
-                if (m_focus_revert_screen != 0) {
-                    revertFocus(*m_focus_revert_screen, false);
-                    m_focus_revert_screen = 0;
-                }
             }
         } else {
             FbTk::Timer::updateTimers(ConnectionNumber(disp)); //handle all timers
@@ -718,52 +712,18 @@ void Fluxbox::handleEvent(XEvent * const e) {
     m_last_event = *e;
 
     // it is possible (e.g. during moving) for a window
-    // to mask certain events to go somewhere (e.g. to that window)
-    if (!m_redirect_events.empty()) {
-
-        bool drop_event = false;
-        RedirectEvents::iterator it = m_redirect_events.begin();
-        RedirectEvents::iterator it_end = m_redirect_events.end();
-        RedirectEvent *re = 0;
-        bool matched = false;
-        Window orig_win = e->xany.window;
-
-        // look through all registered redirects
-        while (it != it_end) {
-            matched = false;
-            re = *it;
-            // do we affect this event?
-            if (e->type == re->catch_type &&
-                (re->catch_win == None ||
-                 re->catch_win == orig_win)) {
-                matched = true;
-                // redirect?
-                if (re->redirect_win != None) {
-                    e->xany.window = re->redirect_win;
-                } else {
-                    drop_event = true;
-                }
-            }
-
-            // does this event stop this redirect?
-            if (e->type == re->stop_type &&
-                ((re->stop_win == None && matched) ||
-                 re->stop_win == orig_win)) {
-                RedirectEvents::iterator next_it = it;
-                ++next_it;
-                delete (*it);
-                m_redirect_events.erase(it);
-                it = next_it;
-            } else
-                ++it;
-        }
-
-        // if one of the redirects says to drop it, we do
-        if (drop_event)
+    // to mask all events to go to it 
+    if ((m_masked == e->xany.window) && m_masked_window) {
+        if (e->type == MotionNotify) {
+            m_last_time = e->xmotion.time;
+            m_masked_window->motionNotifyEvent(e->xmotion);
             return;
+        } else if (e->type == ButtonRelease) {
+            e->xbutton.window = m_masked_window->fbWindow().window();
+        }
+  
     }
 
-
     // update key/mouse screen and last time before we enter other eventhandlers
     if (e->type == KeyPress ||
         e->type == KeyRelease) {
@@ -1211,7 +1171,7 @@ void Fluxbox::handleKeyEvent(XKeyEvent &ke) {
         if (m_watching_screen && m_watch_keyrelease) {
             // mask the mod of the released key out
             // won't mask anything if it isn't a mod
-            ke.state &= ~FbTk::KeyUtil::keycodeToModmask(ke.keycode);
+            ke.state &= ~FbTk::KeyUtil::instance().keycodeToModmask(ke.keycode);
             
             if ((m_watch_keyrelease & ke.state) == 0) {
                 
@@ -1628,6 +1588,7 @@ void Fluxbox::load_rc() {
     else // expand tilde
         *m_rc_stylefile = StringUtil::expandFilename(*m_rc_stylefile);
 
+
     // expand tilde
     *m_rc_groupfile = StringUtil::expandFilename(*m_rc_groupfile);
 
@@ -1934,35 +1895,15 @@ void Fluxbox::setFocusedWindow(WinClient *client) {
  * ignore_event means that it ignores the given event until
  * it gets a focusIn
  */
-void Fluxbox::revertFocus(BScreen &screen, bool wait_for_end) {
+void Fluxbox::revertFocus(BScreen &screen) {
     // Relevant resources:
     // resource.focus_last = whether we focus last focused when changing workspace
     // BScreen::FocusModel = sloppy, click, whatever
-    if (wait_for_end) {
-        if (m_focus_revert_screen == 0) {
-            m_focus_revert_screen = &screen;
-            return;
-        } else if (m_focus_revert_screen == &screen)
-            return;
-        else 
-            cerr<<"Unexpected screen in revertFocus()"<<endl;
-    }
-
-    WinClient *next_focus = 0;
-    long ignore_event = 0;
-    if (screen.doFocusLast()) {
-        next_focus = screen.getLastFocusedWindow(screen.currentWorkspaceID());
-
-        // when doFocusLast is set, we don't do exact sloppy focus - we 
-        // go to the last focused window, rather than the pointer window
-        // i.e. we ignore any EnterNotify events until the focus sending arrives
-        if (screen.getFocusModel() != BScreen::CLICKTOFOCUS)
-            ignore_event = EnterNotify;
-    }
+    WinClient *next_focus = screen.getLastFocusedWindow(screen.currentWorkspaceID());
 
     // if setting focus fails, or isn't possible, fallback correctly
     if (!(next_focus && next_focus->fbwindow() &&
-          next_focus->fbwindow()->setCurrentClient(*next_focus, true, ignore_event))) {
+          next_focus->fbwindow()->setCurrentClient(*next_focus, true))) {
         setFocusedWindow(0); // so we don't get dangling m_focused_window pointer
         switch (screen.getFocusModel()) {
         case BScreen::SLOPPYFOCUS:
@@ -1990,42 +1931,3 @@ void Fluxbox::watchKeyRelease(BScreen &screen, unsigned int mods) {
                   screen.rootWindow().window(), True, 
                   GrabModeAsync, GrabModeAsync, CurrentTime);
 }
-
-/**
- * Allows people to create special event exclusions/redirects
- * useful for getting around X followup events, or for
- * effectively grabbing things
- * The ignore is automatically removed when it finds the wakeup_win
- * with an event matching the wakeup_mask
- * ignore None means all windows
- */
-void Fluxbox::addRedirectEvent(BScreen *screen, 
-                               long catch_type, Window catch_win, 
-                               long stop_type, Window stop_win, 
-                               Window redirect_win) {
-    RedirectEvent * re = new RedirectEvent();
-    re->screen = screen;
-    re->catch_type = catch_type;
-    re->catch_win = catch_win;
-    re->stop_type = stop_type;
-    re->stop_win = stop_win;
-    re->redirect_win = redirect_win;
-
-    m_redirect_events.push_back(re);
-}
-
-// So that an object may remove the ignore on its own
-// stop_type of None means remove all redirects for this window
-void Fluxbox::removeRedirectEvent(long stop_type, Window stop_win) {
-    RedirectEvents::iterator it = m_redirect_events.begin();
-    RedirectEvents::iterator it_end = m_redirect_events.end();
-    RedirectEvent *re = 0;
-    for (; it != it_end; ++it) {
-        re = *it;
-        if (re->stop_win == stop_win && (stop_type == None || stop_type == re->stop_type)) {
-            m_redirect_events.erase(it);
-            delete re;
-            return;
-        }
-    }
-}