-
Notifications
You must be signed in to change notification settings - Fork 63
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
support for scrollbars that are not full client size - proposition #9
Comments
noeldelgado, can you review this change? EDIT: this happens only on Chrome - possible cause: rounding method of css values |
You are right, I just tried and I can confirm. This also happens if you put some offsets for the I also found that the Changing var thumbPositionPercentage = ((offset - thumbHalf) * 100 / this._viewElement.clientHeight);
// to
var thumbPositionPercentage = ((offset - thumbHalf) * 100 / this._scrollbarVerticalElement.offsetHeight); and var thumbPositionPercentage = ((offset - thumbHalf) * 100 / this._viewElement.clientWidth);
// to
var thumbPositionPercentage = ((offset - thumbHalf) * 100 / this._scrollbarHorizontalElement.offsetWidth); Do you want to do the PR? otherwise I can take it. thanks! |
Thanks. Sorry I dont know what is "PR" so better you do it :) |
PR = Pull Request How you ever going into the process of forking and contributing to a project before? If you want to do it I am more than happy to assist you in any way I can. If you do not feel like doing it right now, no problem, I just think it could be a good chance in case you haven't done it before :) |
Thanks but I'm only commenting and submiting bugs. I'm fine with this for now. I have own something like repository when I collect plugins with changes made by me :) |
Ok, cool 👍 |
Now plugin does not support scrollbar width/height that is smaller than clientWidth/clientHeight.
Replace
_mouseMoveDocumentHandler
method with this:The text was updated successfully, but these errors were encountered: