From 75c53faacd67a0d21248f17754ce9225e987f445 Mon Sep 17 00:00:00 2001 From: Cyprien P Date: Mon, 30 Jan 2023 15:13:56 +0100 Subject: [PATCH] (core) Fix document settings' scroll bar Summary: Scroll bar used to show with an offset to the right end of the page. This diff fixes that issue. Test Plan: should not break anything. Reviewers: jarek Reviewed By: jarek Differential Revision: https://phab.getgrist.com/D3778 --- app/client/ui/DocumentSettings.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/app/client/ui/DocumentSettings.ts b/app/client/ui/DocumentSettings.ts index 7f8f4084..61cc247f 100644 --- a/app/client/ui/DocumentSettings.ts +++ b/app/client/ui/DocumentSettings.ts @@ -57,7 +57,7 @@ export class DocSettingsPage extends Disposable { dom.create(buildCurrencyPicker, fromKo(this._currency), (val) => this._currency.saveOnly(val), {defaultCurrencyLabel: t("Local currency ({{currency}})", {currency: getCurrency(l)})}) )), - canChangeEngine ? [ + canChangeEngine ? cssDataRow([ // Small easter egg: you can click on the skull-and-crossbones to // force a reload of the document. cssDataRow(t("Engine (experimental {{span}} change at own risk):", {span: @@ -69,7 +69,7 @@ export class DocSettingsPage extends Disposable { })) })), select(this._engine, getSupportedEngineChoices()), - ] : null, + ]) : null, cssHeader(t('API')), cssDataRow(t("This document's ID (for API use):")), cssDataRow(cssHoverWrapper( @@ -144,7 +144,6 @@ const cssContainer = styled('div', ` position: relative; height: 100%; padding: 32px 64px 24px 64px; - max-width: 487px; @media ${mediaSmall} { & { padding: 32px 24px 24px 24px; @@ -167,6 +166,7 @@ const cssDataRow = styled('div', ` margin: 16px 0px; font-size: ${vars.largeFontSize}; color: ${theme.text}; + width: 360px; `); // Check which engines can be selected in the UI, if any.