-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Breadcrumbs: Add attachment handling
#73249
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
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. |
attachment handling
|
Size Change: +7 B (0%) Total Size: 2.42 MB
ℹ️ View Unchanged
|
mcsf
left a comment
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.
Left comments, but LGTM.
While we're here, could you label the screenshots in the PR description, so that we can tell which is core and which is x3p0?
317a17e to
006954d
Compare
|
Flaky tests detected in 006954d. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/19364242339
|
What?
Part of: #72498
This PR adds
attachmenthandling. The code handles other custom post types similar toattachmentsthat are not hierarchical, but have values inpost_parent.Testing Instructions
wp_attachment_pages_enabledsetting to1. Save changes.View attachment pageScreenshots or screencast
Below I'm sharing some screenshots in a site that I've added x3p0-breadcrumbs plugin and the core block in the main header.
Above is x3p0-breadcrumbs and below is the core block.
With terms
With ancestors
No parent (upload in media library)