From 69e13d782176ca5d2cade4d4327fdf84d7ceb3b4 Mon Sep 17 00:00:00 2001 From: Michael Stapelberg Date: Sun, 6 Nov 2022 19:42:30 +0100 Subject: [PATCH] Revert "gaps: use logical_px() to work correctly on hi-dpi monitors" (#5251) This reverts commit 6b658f88be50d5251f85eaf6f838d6c67ebaac95. The commit was misguided: the pixel values are already run through logical_px() when parsing the configuration directive or command, so they should not be run through another logical_px() pass at rendering time. --- release-notes/changes/1-gaps-hidpi | 1 - src/render.c | 5 ----- 2 files changed, 6 deletions(-) delete mode 100644 release-notes/changes/1-gaps-hidpi diff --git a/release-notes/changes/1-gaps-hidpi b/release-notes/changes/1-gaps-hidpi deleted file mode 100644 index c121e638..00000000 --- a/release-notes/changes/1-gaps-hidpi +++ /dev/null @@ -1 +0,0 @@ -gaps: use logical_px() to scale pixel values to hi-dpi diff --git a/src/render.c b/src/render.c index 61948980..fc2cedd9 100644 --- a/src/render.c +++ b/src/render.c @@ -59,11 +59,6 @@ void render_con(Con *con) { inset.width -= inset.x; inset.height -= inset.y; - inset.x = logical_px(inset.x); - inset.y = logical_px(inset.y); - inset.width = logical_px(inset.width); - inset.height = logical_px(inset.height); - if (con->fullscreen_mode == CF_NONE) { params.rect = rect_add(params.rect, inset); con->rect = rect_add(con->rect, inset);