Skip to content

Remove unused vertical scrollbar gutter - #1064

Closed
jonatanklosko wants to merge 1 commit into
pierrecomputer:mainfrom
jonatanklosko:jk-scrollbar-gutter
Closed

Remove unused vertical scrollbar gutter#1064
jonatanklosko wants to merge 1 commit into
pierrecomputer:mainfrom
jonatanklosko:jk-scrollbar-gutter

Conversation

@jonatanklosko

Copy link
Copy Markdown
Contributor

[data-code] has overflow-y: clip, and overflow-x: scroll, so the effective overflow-y is actually hidden. Consequently, the vertical scrollbar can never appear, but scrollbar-gutter: stable can cause an empty gutter space to be reserved for the vertical scrollbar anyway. It does not always show up, but in our app it does show up in some cases and removing scrollbar-gutter: stable fixes that.

Given that scrollbar-gutter: stable only impacts the inline axis, here the vertical scrollbar, and said scrollbar should never appear, it seems like reserving a gutter is never desirable. I may be missing the original intent of the rule being added though.

@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

@jonatanklosko is attempting to deploy a commit to the Pierre Computer Company Team on Vercel.

A member of the Team first needs to authorize it.

@amadeus

amadeus commented Aug 6, 2026

Copy link
Copy Markdown
Member

It’s very much intentional because it keeps bottom padding consistent around hunk separators.

also it would break calculations in the virtualizers

@amadeus amadeus closed this Aug 6, 2026
@jonatanklosko

Copy link
Copy Markdown
Contributor Author

My understanding is that scrollbar-gutter: stable does not impact horizontal scrollbars though, per the spec:

This property affects the presence of scrollbar gutters placed at the inline start edge or inline end edge of the box.

Does it actually change the spacing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants