Test placeholders with lazy builder

Created on 24 June 2023, about 1 year ago
Updated 4 July 2023, 12 months ago

Problem/Motivation

The only place in standard Drupal where content is rendered after a container is when using lazy building.

Proposed resolution

Test if possible to add placeholder automatically to lazy building placeholder.

Otherwise, add this in the out of scope document.

PS: From Drupal 10.1.0: https://www.drupal.org/node/3338948 β†’ , maybe this would help.

✨ Feature request
Status

Closed: won't fix

Version

5.0

Component

User interface

Created by

πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @Grimreaper
  • πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·
  • Assigned to Grimreaper
  • πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·
  • Issue was unassigned.
  • Status changed to Closed: won't fix 12 months ago
  • πŸ‡«πŸ‡·France Grimreaper France πŸ‡«πŸ‡·

    So, with Drupal 10.1.0, I tried to edit the core/modules/big_pipe/templates/big-pipe-interface-preview.html.twig template to see the result.

    And the problem is that Bootstrap placeholder expects the loading stuff to already take the space on the page while Drupal lazy builder render stuff after, it does not reserve the space on the page.

    Also by default the "preview" variable is empty in the template.

    Like you will see on the screen capture, this would add a lot of flickering to a lot of blocks.

    So let's do nothing and people who want to have a placeholder will have to add some markup using #lazy_builder_preview like mentioned in the change record.

Production build 0.69.0 2024