_bindScroll in infinity-loader.js sets wrong scroll element about ember-infinity

[hhff]

Hi @temo44 !

Thanks for this! Check this link. It explains why we're using the infinity-loader's guid for the event namespace. It's standard jQuery practise to use . to namespace an event. Seeing as a developer will probably be binding other scroll events to the window, we want to be sure to only remove this particular one when the infinity-scroller is destroyed. Make sense?

Also - check out the latest release (just released this morning) #4 - it now has a "scrollable" attribute that can be used to pass in a selector string 😄

[timohermans]

Thank you for the reply @hhff !

It makes total sense. However, won't the selector still be incorrect, as Ember generates infinity-loader's guid as an ID, and not a class?

Cool that the feature is introduced 😄.

Awesome addon btw!

[hhff]

Thanks @temo44 !

The guid is only used as the event namespace - ie: the unique string that "scrollable" (usually the window) uses to identify the event name. It's not a selector - it's a scroll event with a unique namespace.

it could be

Ember.$(scrollElement).on("scroll."+Math.random(), function() {

But a GUID is guaranteed to be unique. make sense?

[timohermans]

Aaah thanks! It totally makes sense now.

I know where my thought process went wrong 😄 ! I still thought it was part of the 'not-scrolling' problem(hence, the idea that it's a selector instead of a namespace), because I have a different scrolling element. Thank you so much!