-
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
Fix: enable link editing interface when removing Draft links #52528
Conversation
Size Change: +24 B (0%) Total Size: 1.43 MB
ℹ️ View Unchanged
|
Flaky tests detected in d46234d. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/5524003800
|
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.
Thanks for the PR 👍
I'm concerned about the API of this component growing with isX
type props.
Also I need to look into the issue in more detail as I'm not sure which route is the most appropriate.
Thank you for taking the time to raise this PR. Ultimately I ended up approved #52415 because I felt it's a better approach and I"d rather avoid any more props being added to Feel free to close if you think that's ok? 🙇 |
What?
Fixes: #50180
Handling a special case in which links to Draft pages in the Navigation Block fail to complete the link removal flow, incorrectly displaying an empty Link Preview.
How?
Uses
forceIsEditingLink
on theLinkControl
to enable or disable the link editing UI depending on whether or not the page was a Draft. If the page is not a Draft, default behavior is unaffected.Testing Instructions
Within the navigation block.