don't use .html() in Core.clean() function

linkesch · linkesch · commit b9e333a767e7 · 2016-03-25T14:41:01.000+01:00
diff --git a/src/js/core.js b/src/js/core.js
@@ -296,11 +296,7 @@
             return;
         }
 
-        // Fix #39
-        // After deleting all content (ctrl+A and delete) in Firefox, all content is deleted and only <br> appears
-        // To force placeholder to appear, set <p><br></p> as content of the $el
-
-        if (this.$el.html().trim() === '' || this.$el.html().trim() === '<br>') {
+        if (this.$el.children().length === 0) {
             this.$el.html(this.templates['src/js/templates/core-empty-line.hbs']().trim());
         }
 

Original file line number	Diff line number	Diff line change
`@@ -296,11 +296,7 @@`
`296`	`296`	`return;`
`297`	`297`	`}`
`298`	`298`
`299`		`- // Fix #39`
`300`		`- // After deleting all content (ctrl+A and delete) in Firefox, all content is deleted and only <br> appears`
`301`		`- // To force placeholder to appear, set <p><br></p> as content of the $el`
`302`		`-`
`303`		`- if (this.$el.html().trim() === '' \|\| this.$el.html().trim() === '<br>') {`
	`299`	`+ if (this.$el.children().length === 0) {`
`304`	`300`	`this.$el.html(this.templates['src/js/templates/core-empty-line.hbs']().trim());`
`305`	`301`	`}`
`306`	`302`