Skip to content

Conversation

@eternicode
Copy link

My project uses a mix of "plain" scripts and requirejs scripts on its pages; plain scripts (jquery, various plugins, etc) are included on every page as part of a "common.js" bundle, and requirejs modules are loaded in some places as the need arises for them.

When scrollTo is included in a script tag, instead of being require()'d, on a page that includes requirejs, requirejs throws an error about a Mismatched anonymous define() module. Adding a name to the module fixes this error for me.

@flesler
Copy link
Owner

flesler commented Apr 3, 2014

I already added this and got several complaints on the same day because that caused others problems. I won't be merging this, you should use a modified version of the plugin sadly :(

@flesler flesler closed this Apr 3, 2014
@eternicode
Copy link
Author

Sad, indeed, but fair enough.

For future reference of those who stumble upon this ticket, #66 discusses why AMD modules shouldn't be named -- apparently that's a feature reserved for r.js's use.

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.

2 participants