-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
First pass at docs for the Widgets Block Editor #31577
Conversation
Size Change: +364 B (0%) Total Size: 1.31 MB
ℹ️ View Unchanged
|
Hey @noisysocks. I don't see that updated in this PR, so I think it may have been overwritten. IIRC, one of the toc.json or the manifest.json is updated manually and the other is auto-generated from that change. Given the manifest changes aren't present, that's probably the one that's auto-generated from the manually updated toc.json. |
Aha, oops, I updated |
…and the wording more consistent and less verbose
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made a few changes, not essential. The docs are great @noisysocks thanks for writing them!
In the "Add a block transform" section of this doc, the example is using a "from" transform. Shouldn't this be a "to" transform, @noisysocks ? |
A If it were a |
Ok. I'm trying to add a transform in a legacy widget I developed years ago and I cannot get it to work either way (from or to) as it doesn't appear in the Transform dropdown. I must be doing something wrong. |
@mrleemon Feel free to share your transform code and I'll see if I can spot anything. |
Here it is:
The block and the legacy widget have two boolean attributes: Thanks in advance |
@mrleemon The blocks array should be The block you're transforming to is indicated by the block definition that the transform appears within (and the block type specified in the This is the reverse of a |
Thanks, that was it! |
Here's my first pass at some docs for the Widgets Block Editor. This checks off many of the items in #25759.
I'm not really familiar with how
docs/manifest.json
works so please let me know if I did something wrong there.