Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make sticky scroll more explicit for new users #215440

Open
aiday-mar opened this issue Jun 13, 2024 · 5 comments
Open

Make sticky scroll more explicit for new users #215440

aiday-mar opened this issue Jun 13, 2024 · 5 comments
Assignees
Labels
editor-sticky-scroll feature-request Request for new features or functionality ux User experience issues

Comments

@aiday-mar
Copy link
Contributor

aiday-mar commented Jun 13, 2024

Sticky scroll is turned on by default for a couple of builds now. We are receiving issues and concerns from users who do not know what the sticky widget is and who confuse it with the editor lines. They think the sticky widget is an editor bug. This issue is about how to handle this issue. Several ideas have been proposed:

  • Turn the sticky widget on by default only when an outline provider is available. For some languages like C# where allman style braces are used, if there are no outline providers, one can see only brackets in the sticky widget, which provides little useful information. To mitigate this, we can introduce a setting which shows the sticky scroll lines only when an outline provider is available. The lines in the sticky widget would then carry in general more useful information.
  • Add a pin on the sticky widget in the far right bottom corner of the sticky widget. When the pin is unpinned (by clicking on it), this would remove the sticky widget. This is a quick way to remove the sticky lines, and indicates to new users this is an individual widget.
  • Instead of adding a pin, we could add a gear icon on the far bottom right, which when clicked, opens the context menu from which it would be possible to toggle the sticky scroll on and off.
  • The sticky widget background color corresponds to the editor background color. This is done for aesthetic purposes. Perhaps we could change the sticky scroll widget background color to something lighter or darker than the default background colors of the base themes.
  • We could also make larger the drop shadow of the sticky widget. This would make it clearer that the sticky widget lies on top of the editor.
  • We could add a tooltip on the sticky widget. When you would hover over the sticky scroll lines, you would see a tooltip with some information like 'Line 14 is the sticky scroll widget'.

Pinging @daviddossett about thoughts, cc @hediet

@aiday-mar aiday-mar self-assigned this Jun 13, 2024
@aiday-mar aiday-mar added feature-request Request for new features or functionality ux User experience issues editor-sticky-scroll labels Jun 13, 2024
@aiday-mar
Copy link
Contributor Author

Hi @daviddossett I was wondering what your thoughts are concerning the above?

@daviddossett
Copy link
Contributor

In general I like the idea of having something like a pin to more clearly convey that metaphor. We should play with a few ideas here. I can find sometime after we chat in the UX sync.

@aiday-mar
Copy link
Contributor Author

The results of running the poll to discuss how sticky scroll should be made more explicit are as follows:

  1. Make the drop-shadow longer with 7 votes
  2. Adding a pin icon with 1 vote
  3. Use only outline model by default, not folding model or indentation model if it does not exist with 1 vote

Given that extending the drop-shadow has amassed the most votes, I will go with this idea for now and we can see later if other measures need to be implemented.

@daviddossett
Copy link
Contributor

Sounds good, I think that's a good place to start and we can tweak from there

@JeremyTBradshaw
Copy link

Throwing my name in the hat for those who are unfamiliar with sticky scroll and do not like it. The thing buggers up in Terminal (WindowsPowerShell/PowerShell, doesn't matter) and you simply cannot get a clean CLS / Clear-Host. I just realized I could hover my mouse over the sticky scroll area and right-click to un-check it being enabled. Would have no idea how to describe the issues caused, and have somebody here understand (way too difficult to describe concisely). I tried once before and was told it's an issue only with Windows 10, not 11, which I never have confirmed. In any case, would totally agree this feature came out of the blue and for me, not knowing anything about it, or wanting it, has just be a pain, which I have gotten clear of by disabling the feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor-sticky-scroll feature-request Request for new features or functionality ux User experience issues
Projects
None yet
Development

No branches or pull requests

3 participants