pekwm / pekwm

pekwm - X11 window manager
GNU General Public License v2.0
120 stars 12 forks source link

Remove latency of PlaceNew #170

Open xa56b345 opened 1 month ago

xa56b345 commented 1 month ago

When opening a new window there's a little latency that make the window show up on the top left of the screen for a second and then apply the PlaceNew value. Would be nice to have no latency and see the window open directly at the correction position

Preview: windowflash.webm

xa56b345 commented 1 month ago

I guess it has to be something related with initial_frame_order from Client.hh

class State {
        public:
                State(void)
                        : maximized_vert(false),
                          maximized_horz(false),
                          shaded(false),
                          fullscreen(false),
                          placed(false),
                          initial_frame_order(0),
                          skip(0),
                          decor(DECOR_TITLEBAR|DECOR_BORDER),
                          cfg_deny(CFG_DENY_NO)
                {
                }
                ~State(void) { }

                bool maximized_vert, maximized_horz;
                bool shaded;
                bool fullscreen;

                // pekwm states
                bool placed;
                uint initial_frame_order; /**< Initial frame position */
                uint skip, decor, cfg_deny;
        } _state;
xa56b345 commented 1 month ago

Tried the new-client-on-create branch work perfectly no more window latency but can't use dmenu/rofi or sometime terminal wont open unusable

xa56b345 commented 1 month ago

For peoples experiencing the same issue all you need to do is to undo this particular commit

https://github.com/pekwm/pekwm/commit/6b0673f29e69659560602d7044ecc9b463397fcd

Work on pekwm-0.3.0

pclouds commented 2 weeks ago

Interestingly the problem I'm experiencing with https://github.com/ValveSoftware/steam-for-linux/issues/9273 seems to be gone with that commit reverted. Thanks!