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

Version 3 plan #27

Open
1 of 4 tasks
hollowdoor opened this issue Apr 25, 2017 · 3 comments
Open
1 of 4 tasks

Version 3 plan #27

hollowdoor opened this issue Apr 25, 2017 · 3 comments

Comments

@hollowdoor
Copy link
Owner

hollowdoor commented Apr 25, 2017

For future refinements, and enhancements in version 3 of dom-autoscroller.

Please use the dev3 branch for developing version 3 in your pull requests.

Put " -version 3" at the end of the title of issues when you:

  • Find bugs that would require a user land breaking fix in version 2
  • Are proposing a major enhancement (even if it's not breaking)
  • Want to work on version 3, and would like a way to track your work
  • Want your proposed enhancement added to the below todo list.
  • Want to forward a bug fix from version 2.

Todo

  • Create independent "safe" dev branch for version 3 (possibly named "dev3")
  • Add release notes file (remove release notes from readme).
  • Remove the boolean switch in the destroy method
  • Work on the syncMove method
    • Should it remain an option, or be a default?
    • Should syncMove be default true, or false?
    • How should syncMove work in the broader environments (touch, ...)
@LeonardoMinatiCrispyBacon

What about a feature to enable only vertical or horizontal scroll? I'm having issues with this because i have absolute positioned elements outside my view.

@hollowdoor
Copy link
Owner Author

I think that would be good. The autoScroll() options method was supposed to be for edge cases like that, but more dedicated options would probably be better because so called "edge cases" might not actually be towards the edge. :)

@LeonardoMinatiCrispyBacon

@hollowdoor That would be great! I tried to solve the problem using the autoscroll callback but couldn't make it work. Maybe i'll branch the project and try it out :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants