From 087159093fbb58043d8d72564d583768031c80b2 Mon Sep 17 00:00:00 2001 From: Fabio Caccamo Date: Thu, 14 Sep 2023 14:17:38 +0200 Subject: [PATCH 1/4] Show scrollbars only if needed. --- src/fontra/client/web-components/designspace-location.js | 2 +- src/fontra/views/editor/editor.css | 2 +- src/fontra/views/editor/panel-designspace-navigation.js | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/fontra/client/web-components/designspace-location.js b/src/fontra/client/web-components/designspace-location.js index eb48964a9..e4de91903 100644 --- a/src/fontra/client/web-components/designspace-location.js +++ b/src/fontra/client/web-components/designspace-location.js @@ -9,7 +9,7 @@ export class DesignspaceLocation extends UnlitElement { display: grid; grid-template-columns: 25% auto; gap: 0.4em; - overflow: scroll; + overflow: auto; } .slider-label { diff --git a/src/fontra/views/editor/editor.css b/src/fontra/views/editor/editor.css index 7fef15835..a20855f08 100644 --- a/src/fontra/views/editor/editor.css +++ b/src/fontra/views/editor/editor.css @@ -334,7 +334,7 @@ body { display: flex; flex-direction: column; gap: 0.4em; - overflow: scroll; + overflow: auto; padding-bottom: 0.4em; } diff --git a/src/fontra/views/editor/panel-designspace-navigation.js b/src/fontra/views/editor/panel-designspace-navigation.js index dee39ed21..0a941ea0b 100644 --- a/src/fontra/views/editor/panel-designspace-navigation.js +++ b/src/fontra/views/editor/panel-designspace-navigation.js @@ -643,7 +643,7 @@ export default class DesignspaceNavigationPanel extends Panel { const locationElement = html.createDomElement("designspace-location", { style: `grid-column: 1 / -1; min-height: 0; - overflow: scroll; + overflow: auto; height: 100%; `, }); From ec2d9da3142f713f0c9053aef4dcc22e07315294 Mon Sep 17 00:00:00 2001 From: Fabio Caccamo Date: Thu, 14 Sep 2023 14:35:54 +0200 Subject: [PATCH 2/4] Remove unused css class. --- src/fontra/views/editor/editor.css | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/fontra/views/editor/editor.css b/src/fontra/views/editor/editor.css index a20855f08..ffd649425 100644 --- a/src/fontra/views/editor/editor.css +++ b/src/fontra/views/editor/editor.css @@ -330,14 +330,6 @@ body { padding: 0.5em; } -.axis-sliders { - display: flex; - flex-direction: column; - gap: 0.4em; - overflow: auto; - padding-bottom: 0.4em; -} - .canvas-container { position: relative; /* for the children */ width: 100%; From fe92ce92aa9b712c594be080b2445cc0e67d2de5 Mon Sep 17 00:00:00 2001 From: Fabio Caccamo Date: Thu, 14 Sep 2023 14:37:56 +0200 Subject: [PATCH 3/4] Show scrollbars only if needed. --- src/fontra/client/web-components/grouped-settings.js | 2 +- src/fontra/client/web-components/ui-form.js | 2 +- src/fontra/filesystem/landing.css | 2 +- src/fontra/views/editor/panel-selection-info.js | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/fontra/client/web-components/grouped-settings.js b/src/fontra/client/web-components/grouped-settings.js index b3694771c..3ae4a3992 100644 --- a/src/fontra/client/web-components/grouped-settings.js +++ b/src/fontra/client/web-components/grouped-settings.js @@ -4,7 +4,7 @@ import { SimpleSettings } from "./simple-settings.js"; export class GroupedSettings extends UnlitElement { static styles = ` :host { - overflow-y: scroll; + overflow-y: auto; } .header { diff --git a/src/fontra/client/web-components/ui-form.js b/src/fontra/client/web-components/ui-form.js index bf4c58657..2ebf45f23 100644 --- a/src/fontra/client/web-components/ui-form.js +++ b/src/fontra/client/web-components/ui-form.js @@ -123,7 +123,7 @@ export class Form extends SimpleElement { grid-template-columns: 32% 68%; gap: 0.35rem 0.35rem; overflow-x: hidden; - overflow-y: scroll; + overflow-y: auto; } .ui-form-label { diff --git a/src/fontra/filesystem/landing.css b/src/fontra/filesystem/landing.css index 716a017eb..65569a0bd 100644 --- a/src/fontra/filesystem/landing.css +++ b/src/fontra/filesystem/landing.css @@ -146,7 +146,7 @@ a { #project-list { display: grid; - overflow-y: scroll; + overflow-y: auto; } .project-item { diff --git a/src/fontra/views/editor/panel-selection-info.js b/src/fontra/views/editor/panel-selection-info.js index 599782dab..e16666e90 100644 --- a/src/fontra/views/editor/panel-selection-info.js +++ b/src/fontra/views/editor/panel-selection-info.js @@ -22,7 +22,7 @@ export default class SelectionInfoPanel extends Panel { height: 100%; width: 100%; overflow-x: hidden; - overflow-y: scroll; + overflow-y: auto; padding: 1em; } `; From 8b4e45632e82c18805c83bf1b0a28a82948d44ef Mon Sep 17 00:00:00 2001 From: Fabio Caccamo Date: Thu, 14 Sep 2023 14:51:38 +0200 Subject: [PATCH 4/4] Show horizontal scrollbars only if needed. --- src/fontra/views/editor/panel-text-entry.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/fontra/views/editor/panel-text-entry.js b/src/fontra/views/editor/panel-text-entry.js index be967e5b7..898620712 100644 --- a/src/fontra/views/editor/panel-text-entry.js +++ b/src/fontra/views/editor/panel-text-entry.js @@ -58,7 +58,7 @@ export default class TextEntryPanel extends Panel { font-family: fontra-ui-regular, sans-serif; font-size: 1.1rem; resize: none; - overflow-x: scroll; + overflow-x: auto; } `;