Skip to content

Conversation

@Cerdic
Copy link

@Cerdic Cerdic commented Nov 13, 2014

TODO : need improvement if script in body part of the page

TODO : need improvement if script in body part of the page
@michael-lynch
Copy link
Owner

Is this necessary? I haven't tested the plugin using a remote page that includes a script but I don't think it should matter, because the remoteTarget will limit the content to the specified element. Can anyone confirm this?

@Cerdic
Copy link
Author

Cerdic commented Dec 9, 2014

The problem is that setTime($('<div>').html(data) make all scripts tags of remote page data beeing executed.
My use case is indicating reading time of "next article", so the remote page is a full HTML page embeding jQuery as the current one. Thus jQuery is overriden with a new instance and this broke some scripts that had a kept some reference to jQuery or some if it's methods.

Copy link

@wilfreddekok wilfreddekok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see a merge conflict?

oops, didn't read @Cerdic 's latest comment.

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.

3 participants