]> git.sur5r.net Git - i3/i3/commitdiff
remove unnecessary code in render_con().
authordarkraven <drdarkraven@gmail.com>
Thu, 6 Sep 2012 15:27:23 +0000 (23:27 +0800)
committerMichael Stapelberg <michael@stapelberg.de>
Sat, 22 Sep 2012 16:13:47 +0000 (18:13 +0200)
The removed code was add by commit 61b8a62 to fix #564. That bug is cause
by rendering the mplayer window again as a floating window (even if it
has been rendered before as a fullscreen window, at line 202). So simply
checking for fullscreen window is enough to solve this problem. Treating
floating/tiling fullscreen window differently is not needed.

src/render.c

index da993a57206ee13fa130616093dbc1e88fd91f38..0eda1a9750aec7b9be2f698431f8875f4ba28c1c 100644 (file)
@@ -246,26 +246,13 @@ void render_con(Con *con, bool render_fullscreen) {
             Con *content = output_get_content(output);
             Con *workspace = TAILQ_FIRST(&(content->focus_head));
 
-            /* Check for (floating!) fullscreen nodes */
+            /* Check for fullscreen nodes */
             /* XXX: This code duplication is unfortunate. Keep in mind to fix
              * this when we clean up the whole render.c */
             Con *fullscreen = NULL;
             fullscreen = con_get_fullscreen_con(workspace, CF_OUTPUT);
-            if (fullscreen) {
-                /* Either the fullscreen window is inside the floating
-                 * container, then we need to render and raise it now… */
-                if (con_inside_floating(fullscreen)) {
-                    fullscreen->rect = output->rect;
-                    x_raise_con(fullscreen);
-                    render_con(fullscreen, true);
+            if (fullscreen)
                     continue;
-                } else {
-                    /* …or it’s a tiling window, in which case the floating
-                     * windows should not overlap it, so we skip rendering this
-                     * output. */
-                    continue;
-                }
-            }
 
             Con *child;
             TAILQ_FOREACH(child, &(workspace->floating_head), floating_windows) {