Merge pull request yabwe#1128 from durgeshahire4/master

yabwe#1127 | Placeholder is visible when only empty table is in Editor.

Author: nmielnik

spec/placeholder.spec.js

@@ -41,6 +41,12 @@ describe('MediumEditor.extensions.placeholder TestCase', function () {
         expect(editor.elements[0].className).not.toContain('medium-editor-placeholder');
     });
 
+    it('should not set a placeholder for elements with table', function () {
+        this.el.innerHTML = '<table></table>';
+        var editor = this.newMediumEditor('.editor');
+        expect(editor.elements[0].className).not.toContain('medium-editor-placeholder');
+    });
+
     it('should set placeholder for elements with empty children', function () {
         this.el.innerHTML = '<p><br></p><div class="empty"></div>';
         var editor = this.newMediumEditor('.editor');

src/js/extensions/placeholder.js

@@ -78,7 +78,7 @@
 
         updatePlaceholder: function (el, dontShow) {
             // If the element has content, hide the placeholder
-            if (el.querySelector('img, blockquote, ul, ol') || (el.textContent.replace(/^\s+|\s+$/g, '') !== '')) {
+            if (el.querySelector('img, blockquote, ul, ol, table') || (el.textContent.replace(/^\s+|\s+$/g, '') !== '')) {
                 return this.hidePlaceholder(el);
             }