Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

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

Comments

Projects
None yet
2 participants
Contributor

Radagaisus commented Oct 16, 2012

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.

Contributor

arlm commented Oct 29, 2012

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

Contributor

Radagaisus commented Oct 29, 2012

@arlm it's on my task list :)

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