-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Dataviews docs: Layout properties checks and link #64918
Conversation
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
- `children`: a list of field's `id` to combine | ||
- `direction`: how should they be stacked, `vertically` or `horizontally` | ||
- `children`: a list of field's `id` to combine | ||
- `direction`: how should they be stacked, `vertically` or `horizontally` |
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.
I mistyped this in the original docs PR, it should be:
- `direction`: how should they be stacked, `vertically` or `horizontally` | |
- `direction`: how should they be stacked, `vertical` or `horizontal` |
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.
Prepared a follow-up PR to fix this typo: #64973
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.
Nice, thank you! I left a comment about direction
values, which I had mistyped in the original PR.
* fix check icons * defaultLayouts clarification and link to properties of layout
What?
Fixes checks in the table and add a link to the table so it can be referenced from other parts of the page
Why?
The table at https://developer.wordpress.org/block-editor/reference-guides/packages/packages-dataviews/#view-object doesn't show properly the checks.
The information of this table is a great reference for other parts of the docs so a link to this part would be helpful.