Problem/Motivation

Same principle as #3518668: Use Fibers for rendering views rows. This will be most useful for multiple value reference fields such as for tags.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3519387

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

catch created an issue. See original summary.

catch’s picture

Status: Active » Needs review
catch’s picture

Status: Needs review » Postponed

Annoyingly this is not achieving what I hoped it would achieve, which is increasing the number of multiple loaded path aliases in conjunction with #3496369: Multiple load path aliases without the preload cache.

This is because the formatter builds a render array with a Url object, that hasn't yet been converted to a URL, so the Url::toString() call actually happens inside the foreach in field.html.twig which is not Fiber aware and won't be.

To make this work we'd need to render the field items before they get to field.html.twig or something like that.

However, the other way to get to the same eventual place is #3081346: Support auto-placeholdering for blocks placed in Layout Builder - if e.g. a term reference field is referenced via a field block in layout builder, then we get the same desired effect as long as other placeholders on the page also render URLs. Similarly with #3518656: Add Dynamic Page Cache and BigPipe support: `block` ComponentSource does not use placeholders/#lazy_builder.

Generally marking this postponed, not sure exactly on what yet.

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.