setContent() with a jquery selector

vancromp · vancromp · commit ff7e7f1c3c08 · 2014-10-12T17:30:43.000+02:00
Hello,

I think there is a little mistake in case of using setContent() with a jquery selector as argument.
It produce a javascript error: "Uncaught TypeError: Cannot read property 'appendTo' of null".
diff --git a/Source/jBox.js b/Source/jBox.js
@@ -1104,7 +1104,7 @@ jBox.prototype.setContent = function(content, ignore_positioning) {
 	// Set the new content
 	switch (jQuery.type(content)) {
 		case 'string': this.content.html(content); break;
-		case 'object': this.content.children().css({display: 'none'}); this.options.content.appendTo(this.content).css({display: 'block'}); break;
+		case 'object': this.content.children().css({display: 'none'}); content.appendTo(this.content).css({display: 'block'}); break;
 	}
 	
 	// Calculate the difference to before the content was set

Original file line number	Diff line number	Diff line change
`@@ -1104,7 +1104,7 @@ jBox.prototype.setContent = function(content, ignore_positioning) {`
`1104`	`1104`	`// Set the new content`
`1105`	`1105`	`switch (jQuery.type(content)) {`
`1106`	`1106`	`case 'string': this.content.html(content); break;`
`1107`		`- case 'object': this.content.children().css({display: 'none'}); this.options.content.appendTo(this.content).css({display: 'block'}); break;`
	`1107`	`+ case 'object': this.content.children().css({display: 'none'}); content.appendTo(this.content).css({display: 'block'}); break;`
`1108`	`1108`	`}`
`1109`	`1109`
`1110`	`1110`	`// Calculate the difference to before the content was set`