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

Add a Minimum scroller height/width #33

Open
Radagaisus opened this issue Oct 16, 2012 · 3 comments
Open

Add a Minimum scroller height/width #33

Radagaisus opened this issue Oct 16, 2012 · 3 comments

Comments

@Radagaisus
Copy link
Contributor

See this screenshot. That tiny little black thing at the right side is antiscroll.

A simple fix is to add a minimal height, and adjust the top position based on it:

//  Scrollbar.Vertical.prototype.update

height = trackHeight * paneHeight / innerEl.scrollHeight
top = trackHeight * innerEl.scrollTop / innerEl.scrollHeight

if height < 30
  top = top - (30 - height)
  height = 30

this.el
      .css('height', height)
      .css('top', top)

Minimum height should probably be an option, as well.

@arlm
Copy link
Contributor

arlm commented Oct 29, 2012

Please, do the fix on a branch of yours and do a pull request or post a diff.

@Radagaisus
Copy link
Contributor Author

@arlm it's on my task list :)

@arlm
Copy link
Contributor

arlm commented Oct 29, 2012

I have merged a lot of code into my antiscroll fork and it contemplates a portion of code to set the minimum size to 20. Take a look on it and when my pull request is accepted/merged we can close this issue, IMHO.

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

No branches or pull requests

2 participants