Horizontal scrollbar in package docs page #944
Comments
github-merge-queue bot
pushed a commit
that referenced
this issue
Feb 10, 2025
In the first three `calc()` expressions, `100vw` is less than `1280px` (otherwise we'd be in the `screen(xl)` media query), so `min(1280px, 100vw)` is just a complex way to say `100vw`, and thus `calc((100vw - (min(1280px, 100vw) - 32px)) / 2 * -1)` is just a very complex way to say `-16px`. I simplified them to the corresponding `-mx-` classes, so that it's clear that they are the same numbers used in `.section-x-inset-xl`. In the last `calc()`, `100vw` is always more than `1280px` (because of the media query), so `min(1280px, 100vw)` is just a complex way of saying `1280px`. I left `1280px - 112px` which is hopefully more clear than just writing `1068px`. Note that using `100vw` instead of `1280px` in the last one would fix the bug reported in #944, but I'm assuming that `min(1280px, 100vw)` is there for a reason so I cannot simply remove it. --- **Edit**: The second commit fixes #944, by using the page width _excluding the vertical scrollbar_ rather than using the full page width to compute the sticky breadcrumb margin.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://jsr.io/@babel-test-6ae45912/[email protected]/doc
I see it both in Chrome and Firefox
It's only when the page is larger than 1280px.
The problem is the
-section-x-inset-xlclass used in the docs header. However, I'm not going to submit a PR to fix that because it looks like it's a bunch of hard-coded numbers and I don't know where they are coming from.The text was updated successfully, but these errors were encountered: