IScroll is undefined

[shynome]

when I use requirejs load fullpage & scrolloverflow , it appears.
But I can't found where fullpage require 'scrolloverflow' or other name to use it

[alvarotrigo]

Duplicated of #2250

The problem when using require is that makes dependencies compulsory as far as I know.
And scrollOverflow shouldn't be compulsory.

It seems to be some kind of trick, but I didn't look at it very much to be honest.

I'll close the issue to keep a single open threat about it. (#2250)

Feel free to create a pull request to solve it.

[shynome]

Thanks

Álvaro notifications@github.com于2016年9月22日周四 下午5:06写道：

Duplicated of #2250
#2250

The problem when using require is that makes dependencies compulsory as
far as I know.
And scrollOverflow shouldn't be compulsory.

It seems to be some kind of trick
http://stackoverflow.com/questions/14164610/requirejs-optional-dependency,
but I didn't look at it very much to be honest.

I'll close the issue to keep a single open threat about it. (#2250
#2250)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#2281 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQg4ywxMBzbiWb7aUH7STDqJ0KMQTJnQks5qskUSgaJpZM4KDlpd
.

[ghost]

scrolloverflow is amd compatible and can be used with requirejs. But fullpage does wrong define definition for amd. it defines jquery as dependency but not scrolloverflow. Therefore it is unknown within define and factory. To fix this issue for amd the fullpage.js needs to be refactored therefore.

[alvarotrigo]

But fullpage does wrong define definition for amd. it defines jquery as dependency but not scrolloverflow.

If you read my previous answer you'll see why's that.

The problem when using require is that makes dependencies compulsory as far as I know.
And scrollOverflow shouldn't be compulsory.