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 issue with vertical scrollbar not going all the way down #536

Merged
merged 2 commits into from
Oct 14, 2020

Conversation

connyekholm
Copy link
Contributor

Our company use the slickgrid and we had an issue with the vertical scrollbar not going all the way down in some scenarios. There is a gap at the bottom and there is no way to drag the scrollbar all the way down. The horizontal scrollbar is visible and the fault is that the code does not acknowledge this. There is a line in the code that determines if the horizontal scrollbar is visible but this misses with one pixel. The changed formula uses greater than or equal instead of only greater than. I saw that this solution was proposed in a PR long time ago to mleibman:
https://github.com/mleibman/SlickGrid/pull/1030/files

@ghiscoding
Copy link
Collaborator

That seems to not change much on my side, so I guess it's ok
But your PR is very hard to review since you have put the auto-format on save, the entire file is being reformatted and it's hard to know what really changed (unless we click on "hide whitespace changes"). You might have to redo it without auto-format

@ghiscoding
Copy link
Collaborator

@6pac
I'm ok with the change, it has no impact on my side.

@ghiscoding ghiscoding requested a review from 6pac October 5, 2020 18:30
@ghiscoding
Copy link
Collaborator

@6pac
I'm ready for another release after merging this PR, got a few fixes done.

@6pac 6pac merged commit 8c16896 into 6pac:master Oct 14, 2020
@6pac
Copy link
Owner

6pac commented Oct 14, 2020

OK, shall I do a release now?

@ghiscoding
Copy link
Collaborator

Yes that would be great, thanks mate

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.

3 participants