Bug and feature request about scrollbar #6876

Closed
opened 2026-01-31 00:49:32 +00:00 by claunia · 8 comments
Owner

Originally created by @cloudhan on GitHub (Mar 14, 2020).

With scrollbarState hidden, there will be different visual effect

With Windows PowerShell, "scrollbarState" not set (this is a normal case, both bar and space present):
win-ps-visible

With Windows PowerShell, "scrollbarState" set to "hidden" (abnormal):
The scroll bar is hidden, but the space is left there
win-ps-hidden

With PowerShell Core (this is a normal case, the bar and the space is hidden):
pscore-hidden

The bug is the abnormal case, I think it might be related with #3878.
The feature request is add ability to hide the space but show the bar.

Originally created by @cloudhan on GitHub (Mar 14, 2020). With scrollbarState hidden, there will be different visual effect With Windows PowerShell, "scrollbarState" not set (this is a normal case, both bar and space present): ![win-ps-visible](https://user-images.githubusercontent.com/1279292/76679518-28550780-661c-11ea-96fc-3b25a0140360.png) With Windows PowerShell, "scrollbarState" set to "hidden" (abnormal): The scroll bar is hidden, but the space is left there ![win-ps-hidden](https://user-images.githubusercontent.com/1279292/76679523-3acf4100-661c-11ea-8c5e-88f3cd78acaa.png) With PowerShell Core (this is a normal case, the bar and the space is hidden): ![pscore-hidden](https://user-images.githubusercontent.com/1279292/76679538-62bea480-661c-11ea-99e1-e675083f26aa.png) The bug is the abnormal case, I think it might be related with #3878. The **feature request** is add ability to hide the space but show the bar.
Author
Owner

@cloudhan commented on GitHub (Mar 14, 2020):

Forgot to mention, Terminal Version: 0.9.433.0

@cloudhan commented on GitHub (Mar 14, 2020): Forgot to mention, Terminal Version: 0.9.433.0
Author
Owner

@DHowett-MSFT commented on GitHub (Mar 16, 2020):

I believe we accidentally fixed this with #4729 -- when you set scrollbarState it doesn't reload scrollbars that already exist.

@DHowett-MSFT commented on GitHub (Mar 16, 2020): I believe we accidentally fixed this with #4729 -- when you set `scrollbarState` it doesn't reload scrollbars that already exist.
Author
Owner

@DHowett-MSFT commented on GitHub (Mar 16, 2020):

(This will be fixed in the next release.)

@DHowett-MSFT commented on GitHub (Mar 16, 2020): (This will be fixed in the next release.)
Author
Owner

@DHowett-MSFT commented on GitHub (Mar 16, 2020):

(I didn't see your feature request; that one I'm going to keep closed as we've determined it's fairly difficult to do custom scrollbar things and the state of the art here is acceptable.)

@DHowett-MSFT commented on GitHub (Mar 16, 2020): (I didn't see your feature request; that one I'm going to keep closed as we've determined it's fairly difficult to do custom scrollbar things and the state of the art here is acceptable.)
Author
Owner

@cloudhan commented on GitHub (Mar 17, 2020):

@DHowett-MSFT I don't see how this feature can be difficult, isn't the scrollbar just an overlay over the text?

@cloudhan commented on GitHub (Mar 17, 2020): @DHowett-MSFT I don't see how this feature can be difficult, isn't the scrollbar just an overlay over the text?
Author
Owner

@DHowett-MSFT commented on GitHub (Mar 18, 2020):

@cloudhan There's a bit more information about how annoying doing custom scrollbar stuff is in https://github.com/microsoft/terminal/issues/3454#issuecomment-550329209

@DHowett-MSFT commented on GitHub (Mar 18, 2020): @cloudhan There's a bit more information about how annoying doing custom scrollbar stuff is in https://github.com/microsoft/terminal/issues/3454#issuecomment-550329209
Author
Owner

@cloudhan commented on GitHub (Mar 18, 2020):

I actually achived what I want with a negative padding, quiet ugly 😂

@cloudhan commented on GitHub (Mar 18, 2020): I actually achived what I want with a negative padding, quiet ugly 😂
Author
Owner

@DHowett-MSFT commented on GitHub (Mar 18, 2020):

LOL. I didn't even know that worked.

@DHowett-MSFT commented on GitHub (Mar 18, 2020): LOL. I didn't even know that worked.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#6876