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

Fix horizontal scrollbar not to disappear #8898

Merged
merged 1 commit into from
Jan 13, 2021

Conversation

kena0ki
Copy link
Contributor

@kena0ki kena0ki commented Dec 23, 2020

Signed-off-by: Ken Aoki [email protected]

What it does

Fixes #8886.

The module perfect-scrollbar unnecessarily moves the horizontal scrollbar to right. This PR set position attribute for the horizontal scrollbar to static so that it can't be moved.

This PR adds position: relative to style for the scrollbar container as per perfect-scrollbar README.

How to test

  1. Open as many files as the scrollbar appears.
  2. Scroll to right.
  3. Scrollbar should not disappear (not overflow right).

scrollbarfix2

Review checklist

Reminder for reviewers

@kena0ki
Copy link
Contributor Author

kena0ki commented Dec 26, 2020

I found position: static also makes bottom unavailable, so another solution is needed.

@kena0ki kena0ki marked this pull request as draft December 26, 2020 03:53
@kena0ki kena0ki marked this pull request as ready for review December 26, 2020 05:34
@kena0ki
Copy link
Contributor Author

kena0ki commented Dec 26, 2020

I changed the way to fix.

@vince-fugnitto vince-fugnitto added bug bugs found in the application ui/ux issues related to user interface / user experience labels Jan 4, 2021
Copy link
Contributor

@DucNgn DucNgn left a comment

Choose a reason for hiding this comment

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

LGTM 👍

I confirm that:

  • with many opening files on the tab bar, the scrollbar doesn't disappear when either scroll to right or to left.
  • resize the application window, the scrollbar still works as expected.
  • resize editor view (horizontally and vertically), the scrollbar still appears when scrolling to both ends.

OS: macOS

Copy link
Contributor

@colin-grant-work colin-grant-work left a comment

Choose a reason for hiding this comment

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

This looks good to me. This feature of perfect scrollbar causes a lot of pain - it's easy to forget to set the position!

@kena0ki
Copy link
Contributor Author

kena0ki commented Jan 13, 2021

@dukengn @colin-grant-work Thank you for the review.

Is there anything I need to do for this PR to be merged?
I believe CI failure is not related with this PR.

@colin-grant-work colin-grant-work merged commit 1c84263 into eclipse-theia:master Jan 13, 2021
@colin-grant-work
Copy link
Contributor

@kena0ki, good to go, and now done.

@kena0ki
Copy link
Contributor Author

kena0ki commented Jan 13, 2021

@colin-grant-work Thank you!

@kena0ki kena0ki deleted the issue8886 branch January 14, 2021 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application ui/ux issues related to user interface / user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Horizontal scroll bar under tabs can disappear
4 participants