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.

Sign up for GitHub

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

Jump to bottom

Scrollbar slider should have a min height #35116

Closed
alexdima opened this issue Sep 26, 2017 · 1 comment · Fixed by #41385
Closed

Scrollbar slider should have a min height #35116

alexdima opened this issue Sep 26, 2017 · 1 comment · Fixed by #41385
Assignees
@Tyriar
Labels
bug Issue identified by VS Code Team member as probable bug good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities terminal Integrated terminal issues verified Verification succeeded
Milestone
January 2018

Comments

@alexdima
Copy link
Member

Testing #34868

emoji-test.txt

  • save this file as UTF8 with bom (I think it is already)
  • configure "terminal.integrated.scrollback": 5000
  • run chcp 65001
  • run cat emoji-test.txt
  • Obseve: the scrollbar slider becomes very small and it is difficult to grab it with the mouse
  • It would be nice if the mouse cursor would not be an I or if it would give hover feedback:

slider
@vscodebot vscodebot bot added the editor label Sep 26, 2017
@alexdima alexdima removed the editor label Sep 26, 2017
@alexdima alexdima removed their assignment Sep 26, 2017
@alexdima alexdima added the terminal Integrated terminal issues label Sep 26, 2017
@Tyriar Tyriar added this to the Backlog milestone Sep 26, 2017
@Tyriar Tyriar added the ux User experience issues label Sep 26, 2017
@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities and removed ux User experience issues labels Dec 28, 2017
@YannickMeeus
Copy link
Contributor

I'd like to pick this up, if possible.

@YannickMeeus YannickMeeus mentioned this issue Jan 10, 2018
Merged
@Tyriar Tyriar modified the milestones: Backlog, January 2018 Jan 12, 2018
@aeschli aeschli added the verified Verification succeeded label Feb 1, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Feb 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Tyriar Tyriar

Labels
bug Issue identified by VS Code Team member as probable bug good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities terminal Integrated terminal issues verified Verification succeeded
Projects
None yet
Milestone
January 2018
Development

Successfully merging a pull request may close this issue.

Set the minimum height of the xterm scrollbar thumb to 35px
4 participants
@YannickMeeus @Tyriar @alexdima @aeschli