Proposal: Add placeholder text to remind users to fill it in

[candy02058912]

Related to #57
This is a proposal since we're not able to use metadata/bindings.

Walkthrough (with audio)

Demo.mp4

[bacoords]

Thanks! I know that placeholders are also an issue in core and are going to be (hopefully) fixed in the next GB release and make it into 6.7.

I also noticed that sometimes this is an issue for me and sometimes it's not, so I want to understand to see why that is.

The only issue I see is that we (and core) might actually want to allow empty post meta fields in bindings, right? Couldn't a user want to leave a field empty?

That said, if this isn't resolved upstream, I think this PR is a good middleground because non-editable fields is not good! So I'd say we merge and if it gets solved in GB then we can always remove

[bacoords]

Related to #63

[zaguiini]

I love the idea, and I think it's in the right direction.

What if we did it slightly different: instead of asking them to put a placeholder, if the value is empty, we fill it with the block variation name? We could have some different defaults depending on the block. For example, this wouldn't work for Image. We could fill a default value for ID and url, just so it's not ever empty.

Another possibility is we always set a default value for the post meta field. I thought this was already happening, but apparently it isn't. I'll check.

EDIT: While I was thinking about it, why not simply enforce a default value before letting them bind the block? This way, we'll always have a value. I think this is the easiest way, especially if we're talking about handling Images and Buttons.

[autumnfjeld]

The only issue I see is that we (and core) might actually want to allow empty post meta fields in bindings, right? Couldn't a user want to leave a field empty?

Hpefully we find this out from people using the plugin. It could be interesting to have a page linked in the readme "Things we'd like to know."

[autumnfjeld]

This would be the "minimum" good solution. It is better than nothing. It clearly indicates "this is a bound block and needs a placeholder value."

But better would be if @zaguiini 's ideas can be done.