Scrollbar is inset by padding and probably shouldn't be #1810

Closed
opened 2026-01-30 22:37:01 +00:00 by claunia · 4 comments
Owner

Originally created by @mcpiroman on GitHub (Jun 21, 2019).

Environment

commit 08464648

Steps to reproduce

In settings, set padding upper, lower or right to be > 0

Expected behavior

Only text should be padded (I think)

Actual behavior

Scrollbar is padded too

Originally created by @mcpiroman on GitHub (Jun 21, 2019). # Environment commit 08464648 # Steps to reproduce In settings, set padding upper, lower or right to be > 0 # Expected behavior Only text should be padded (I think) # Actual behavior Scrollbar is padded too
Author
Owner

@yodurr commented on GitHub (Jun 22, 2019):

Shouldn't the scrollbar match the text it scrolls? Do something extreme like 100 vertical padding and it looks pretty clear to me the scrollbar should match the padding settings the way it does now.

@yodurr commented on GitHub (Jun 22, 2019): Shouldn't the scrollbar match the text it scrolls? Do something extreme like 100 vertical padding and it looks pretty clear to me the scrollbar should match the padding settings the way it does now.
Author
Owner

@mdtauk commented on GitHub (Jun 22, 2019):

I would have thought the ScrollBar should always align to the edge of the view, and the padding is for the content within it.

@mdtauk commented on GitHub (Jun 22, 2019): I would have thought the ScrollBar should always align to the edge of the view, and the padding is for the content within it.
Author
Owner

@Stanzilla commented on GitHub (Jun 23, 2019):

Yeah it should always align to the edge. Currently it also gets moved off-screen when you set a left padding

@Stanzilla commented on GitHub (Jun 23, 2019): Yeah it should always align to the edge. Currently it also gets moved off-screen when you set a left padding
Author
Owner

@ghost commented on GitHub (Aug 3, 2019):

:tada:This issue was addressed in #1778, which has now been successfully released as Windows Terminal Preview v0.3.2142.0.🎉

Handy links:

@ghost commented on GitHub (Aug 3, 2019): :tada:This issue was addressed in #1778, which has now been successfully released as `Windows Terminal Preview v0.3.2142.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v0.3.2142.0) * [Store Download](https://www.microsoft.com/store/apps/9n0dx20hk701?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#1810