vscode icon indicating copy to clipboard operation
vscode copied to clipboard

Add UI for toggling sticky scroll

Open gjsjohnmurray opened this issue 1 year ago • 1 comments

Type: Feature Request

There is a bit of space at the start of the breadcrumb which could accommodate a pin / pinned codicon with which to toggle the stickyScroll setting.

image

When no DocumentSymbolProvider is available the space could either be blank or could show the pin codicon if the setting is currently false and the pinned-dirty if true, with a hover tip explaining why sticky scrolling isn't happening for the current document.

/assign @aiday-mar

VS Code version: Code - Insiders 1.71.0-insider (dced70bbf36d3c53c08e791da1791ac7fc42519b, 2022-08-10T05:17:53.560Z) OS version: Windows_NT x64 10.0.22000 Modes: Sandboxed: Yes

gjsjohnmurray avatar Aug 10 '22 10:08 gjsjohnmurray

Thanks for the idea, I will bring it up in the UX Sync today

aiday-mar avatar Aug 10 '22 10:08 aiday-mar

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

VSCodeTriageBot avatar Aug 10 '22 13:08 VSCodeTriageBot

Screenshot 2022-08-16 at 10 40 37

I would go with the more traditional entry in the "View"-menu first, similar to existing action. The pin/unpin icon in breadcrumbs might be seen as noisy for when you don't want to use sticky scroll (appears like an ad)

@aiday-mar This is how it is done for the breadcrumbs toggle

We can also combine this with https://github.com/microsoft/vscode/issues/156457 and add context menu entries for breadcrumbs and sticky-scroll to toggle them in-place

jrieken avatar Aug 16 '22 08:08 jrieken

I'll look into this

aiday-mar avatar Aug 16 '22 12:08 aiday-mar