Fix placeholder over media embed.

src/view/placeholder.js

@@ -151,6 +151,12 @@ export function needsPlaceholder( element ) {
 		return false;
 	}
 
+	// If the element is a Widget, always consider it a non-empty and thus not needing a placeholder.
+	// https://github.com/ckeditor/ckeditor5/issues/1684
+	if ( element.getCustomProperty( 'widget' ) ) {
+		return false;
+	}
+
 	// The element is empty only as long as it contains nothing but uiElements.
 	const isEmptyish = !Array.from( element.getChildren() )
 		.some( element => !element.is( 'uiElement' ) );

tests/view/placeholder.js

@@ -389,6 +389,20 @@ describe( 'placeholder', () => {
 			expect( needsPlaceholder( element ) ).to.be.false;
 		} );
 
+		it( 'should return false if element is a widget', () => {
+			// Widget with ui-only elements inside (emptyish).
+			setData( view, '<div><ui:span></ui:span></div>' );
+			viewDocument.isFocused = false;
+
+			const element = viewRoot.getChild( 0 );
+
+			view.change( writer => {
+				writer.setCustomProperty( 'widget', true, element );
+			} );
+
+			expect( needsPlaceholder( element ) ).to.be.false;
+		} );
+
 		it( 'should return true if element is empty and document is blurred', () => {
 			setData( view, '<p></p>' );
 			viewDocument.isFocused = false;