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

Scrollbar size settings in preference implemented #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EstFoisy
Copy link
Owner

@EstFoisy EstFoisy commented Nov 17, 2022

What it does

When the horizontal and vertical scrollbar sizes are adjusted in setting, it should reflect on the scrollbar height in the editor.

Closes eclipse-theia#11732

How to test

  1. Open a new file
  2. Move the cursor to the right until the bottom scrollbar appear
  3. Move the cursor down until the side scrollbar appear
  4. Then go into file: preference: Open Setting
  5. In the top tab bar type: scrollbar size
  6. Change the value of the horizontal and vertical scrollbar size.
  7. Go back into the file, move the cursor and the scrollbar should adjust according to the width and height of the preference.

**please note these change only apply to the scrollbar of the editor.

Issue.11732.-before.corrections.mp4
Issue.11732.-.after.corrections.mp4

Review checklist

  • As an author, I have thoroughly tested my changes and carefully followed [the

review guidelines](https://github.com/theia-ide/theia/blob/master/doc/pull-requests.md#requesting-a-review)

Reminder for reviewers

Comment on lines 141 to 142
verticalScrollbarSize: 10,
horizontalScrollbarSize: 10

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we removing the defaults?

When the horizontal and vertical scrollbar sizes are adjusted in setting
 (in the preference menu), it now apply the scrollbar height or width in
 the editor.
Copy link

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tested but the changes look good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scrollbar size settings are "dendrified"
2 participants