[docs] Allow sidebar to be scrolled on short screens (#1632)
This PR changes how the sidebar works on the docs site. ## What's the problem? Previously, you couldn't reach the bottom of the sidebar on some screens - unless you scroll to the very bottom of the main content. Some of our pages are [very long](https://canary.tldraw.dev/gen/editor/Editor-class), so this isn't practical. ![2023-06-22 at 11 25 26 - Violet Gazelle](https://github.com/tldraw/tldraw/assets/15892272/873ffa98-5eea-48b4-a005-c3aa89074065) ## How are we fixing it? This PR makes the sidebar scroll separately from the main content. It's less fancy and less pretty :/ But it's usable. Maybe we could make it sleeker somehow, but this works at least. ![2023-06-22 at 11 27 29 - Rose Hedgehog](https://github.com/tldraw/tldraw/assets/15892272/99bd47b9-d07c-46ff-b1bb-de0ac289d294) ## Admin ### Change Type - [x] `documentation` — Changes to the documentation only[^2] [^1]: publishes a `patch` release, for devDependencies use `internal` [^2]: will not publish a new version ### Test Plan 1. On the docs site, make your browser window short so that you can't see the whole sidebar. 2. Make sure that you can scroll down to the bottom on the sidebar, separately from the rest of the page. ### Release Notes - Documentation: Fixed the sidebar being unscrollable on some short screens.
This commit is contained in:
parent
004787d5bd
commit
0c046a561b
1 changed files with 3 additions and 2 deletions
|
@ -103,7 +103,6 @@ body {
|
|||
column-gap: 28px;
|
||||
row-gap: 0px;
|
||||
margin: 0px auto;
|
||||
padding-top: 40px;
|
||||
}
|
||||
|
||||
.icon-button {
|
||||
|
@ -168,7 +167,7 @@ body {
|
|||
}
|
||||
|
||||
.article {
|
||||
padding: 16px 16px 120px 16px;
|
||||
padding: 56px 16px 120px 16px;
|
||||
font-weight: 400;
|
||||
max-width: 100%;
|
||||
overflow-x: hidden;
|
||||
|
@ -459,8 +458,10 @@ body {
|
|||
padding: 8px;
|
||||
align-self: start;
|
||||
top: 0px;
|
||||
max-height: calc(100vh);
|
||||
z-index: 1000;
|
||||
overflow-y: auto;
|
||||
padding-top: 40px;
|
||||
}
|
||||
|
||||
.sidebar .sidebar__list h4 {
|
||||
|
|
Loading…
Reference in a new issue