From 4cd4c2f60734f462aceab356e36c8d21b757eea1 Mon Sep 17 00:00:00 2001 From: Papierkorb2292 <104673791+Papierkorb2292@users.noreply.github.com> Date: Sat, 9 Aug 2025 16:31:56 +0200 Subject: [PATCH] Apply suggestions from code review Use `mobile` class on body to determine the style of `CloseZenButton` instead of an extra class Co-authored-by: Elian Doran --- apps/client/src/widgets/close_zen_button.ts | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/apps/client/src/widgets/close_zen_button.ts b/apps/client/src/widgets/close_zen_button.ts index e5842dc14..7ee4fcc28 100644 --- a/apps/client/src/widgets/close_zen_button.ts +++ b/apps/client/src/widgets/close_zen_button.ts @@ -29,7 +29,7 @@ const TPL = /*html*/`\ -webkit-app-region: no-drag; } - body.zen .mobile-close-zen-container { + body.zen.mobile .close-zen-container { top: -2px; } @@ -45,7 +45,6 @@ export default class CloseZenButton extends BasicWidget { doRender(): void { this.$widget = $(TPL); - this.$widget.toggleClass("mobile-close-zen-container", utils.isMobile()) } zenChangedEvent() {