Can't pass React component as placeholder anymore

[czechdave]

Do you want to request a feature or report a bug?

BUG

What's the current behavior?

placeholder prop on <Editor /> can't be a React component anymore. If it's not a string the editor crashes.

What's the expected behavior?

The <Editor /> just renders placeholder if it's a React component.

https://github.com/ianstormtaylor/slate/blob/master/packages/slate-react-placeholder/src/index.js#L22

[kilrain]

I brought this up in the slack channel recently: https://slate-js.slack.com/archives/C1RH7AXSS/p1564944468254200

[gracicot]

To workaround this issue we decided to re-implement the placeholder plugin. We needed the ability to render a react element because our placeholder was allowing text wrapping (we have a whole welcome message in it).

To allow the text wrapping we used absolute positionning and a relative element, which both have different styling properties. Doing that without react elements is not possible as far as I know.

[ianstormtaylor]

Fixed by #3093.