diff --git a/CHANGELOG.md b/CHANGELOG.md index 1a59bba5..9d4a4a34 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -86,6 +86,12 @@ ### Deprecated ### Removed + +* Workaround for slow resize in Sway <= 1.5, when a foot window was + hidden, for example, in a tabbed view + (https://codeberg.org/dnkl/foot/pulls/507). + + ### Fixed * Font size adjustment (“zooming”) when font is configured with a diff --git a/wayland.c b/wayland.c index 725841e2..bac94f48 100644 --- a/wayland.c +++ b/wayland.c @@ -741,43 +741,6 @@ xdg_surface_configure(void *data, struct xdg_surface *xdg_surface, xdg_surface_ack_configure(xdg_surface, serial); -#if 1 - /* - * This was done as a workaround for a bug in Sway - * (https://github.com/swaywm/sway/issues/6023 - * - * Sway waited for configure ACKs from hidden windows. These never - * arrived because Sway never scheduled any frame callbacks for - * these hidden windows... - * - * This has bene fixed as of Sway-1.6. - * - * Unfortunately, slow moving distros (Debian *cough* are still - * stuck on Sway-1.5, and will be so for the foreseeable - * future. - * - * TODO: check for e.g. SWAYSOCK and only preempt if it exists? - * We'd still be pre-emptying on Sway-1.6, but at least other - * compositors would be unaffected. - * - * Or do we ignore Sway-1.5 users and point them to the Sway bug? - * - * Note that it doesn't appear to cause any issues to have this - * enabled, on any compositor. - */ - if (term->window->frame_callback != NULL) { - /* - * Preempt render scheduling. - * - * Each configure event require a corresponding new - * surface+commit. Thus we cannot just schedule a pending - * refresh if there’s already a frame being rendered. - */ - wl_callback_destroy(term->window->frame_callback); - term->window->frame_callback = NULL; - } - -#endif #if 1 /* * TODO: decide if we should do the last “forced” call when ending