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

Option to only scroll up and option to set an offset to the final position #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

oleics
Copy link

@oleics oleics commented Apr 8, 2013

We need two features:

Scroll one item of a long list to the top of the browser window - and only to the top of it. I called the option .alignWithTop with respect to the native .scrollIntoView() and its alignWithTop-argument. The type of .alignWithTop is boolean.

The second feature is an option to set some offset to the position to scroll to. I called the option .offset. The type of .offset is number || integer.

This PR implements both.

@Arwid
Copy link
Owner

Arwid commented Apr 9, 2013

Great idea. .offset is useful. For .alignWithTop, it would be nice for it to fully imitate what it does in the native function; not only align to top if true, but align to bottom if false, and to force the alignment even if the element is already in view. With that in mind, something like {align:"top|bottom|auto", force:true|false} could do the trick, where {align:"auto", force:false} would be the default configuration and {align:"top", force:true} would match the native function. Thoughts?

@dotnetwise
Copy link

Is this ever being considered?

@pherrymason
Copy link
Contributor

+1

1 similar comment
@qstosh
Copy link

qstosh commented Jun 27, 2014

+1

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.

5 participants