-
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
Link modal suggestions only include posts when they should also include pages and CPTs #2084
Comments
For this, do note that you can make the request with |
I'd suggest to milestone https://core.trac.wordpress.org/ticket/39965 (add search endpoint) for 5.0. |
This ticket was mentioned in Slack in #core-editor by jeffpaul. View the logs. |
In the current editor it is also possible to inject non-WordPress resources with the |
@felixarntz is going to do a first pass on this against Gutenberg over the next week. |
…ntrol_on_heading_for_android Disable align controls for the Heading block in Android
The link modal links suggestions implemented in #1985 currently searches just for posts.
Instead, the current implementation in WordPress searches also for pages and Custom Post Types (any public post):
As mentioned in #1985, this is a limitation of the REST API. See the related Trac ticket: https://core.trac.wordpress.org/ticket/39965. Regardless of the reason why, right now there's no feature parity with the current WordPress implementation.
There are also performance concerns, since by default the REST API returns all the post fields including the post content. The response can be huge in size, even several hundreds KB or more than 1MB with very long posts. See related considerations on:
https://core.trac.wordpress.org/ticket/38920
https://core.trac.wordpress.org/ticket/38922
and especially:
"REST API: Provide interface to include or exclude specific fields from response JSON"
https://core.trac.wordpress.org/ticket/38131
The text was updated successfully, but these errors were encountered: