(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
This commit is contained in:
Cyprien P 2023-01-30 15:13:56 +01:00
parent 3be48c9a7e
commit 75c53faacd

View File

@ -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.