Skip to content

Commit

Permalink
Theme: Increase page max-width from 1040px to 1100px
Browse files Browse the repository at this point in the history
At wide viewports, of the 60px extra space 50px goes to the sidebar
and 5px goes to the content area. This is motivated by improving the
sidebar to reduce wrapping of page titles, page groups, and TOC headings.

At narrow viewports, we now prioritize the content over the sidebar,
letting the sidebar shrink a bit.

Narrow 768px
* Before: 220px sidebar    , 470px content
* After:  185px sidebar (-), 505px content (+)

Wide
* Before (1040px): 220px sidebar     , 757px content
* After  (1100px): 275px sidebar (++), 762px content (+)
  • Loading branch information
Krinkle committed Jan 16, 2025
1 parent a1ed520 commit 67ddfa4
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions _sass/amethyst.scss
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ iframe {
}

.wrapper {
max-width: 65rem;
max-width: 1100px;
margin: 0 auto;
@media (min-width: $screen-m) {
padding: 0 $size-spacing;
Expand All @@ -159,12 +159,10 @@ iframe {
}

// Allow layouts to have a sidebar
.sidebar {
display: none;
}
@media (min-width: $screen-m) {
.main--columns {
display: flex;
gap: 20px;
}
.content {
// Take up available space even if the lines of text are short.
Expand All @@ -182,8 +180,11 @@ iframe {
.sidebar {
order: -1;
display: block;
flex: 0 0 220px;
margin-right: 20px;
// between 185px and 275px
// =26% of 768px (-2x21px padding, -16px scrollbar)
// =26% of 1100px (-2x21px padding)
flex: 0 0 26%;
background-image: radial-gradient(ellipse at right, $color-light, $color-white 80%);
}
}

Expand Down

0 comments on commit 67ddfa4

Please sign in to comment.