2. Add {% section %} tag to determine empty-ness of a region without rendering twice (non-placeholder case)

Created on 13 October 2016, almost 9 years ago
Updated 21 August 2025, about 2 months ago

Problem/Motivation

In 🌱 [meta] Themes improperly check renderable arrays when determining visibility Needs work the second case is that a region does contain empty blocks, but no placeholders.

It must be rendered (and html tags stripped?) to determine if it is empty or not, but we don't want to render it twice.

Proposed resolution

Use a new twig tag, so instead of:

Old:

{% if content.sidebar_first %}
<div class="sidebar-first">
  {{ content.sidebar_first }}
</div>
{% endif %}

use new:

{% section content.sidebar_first %}
{# This whole section will only be shown if the passed variable is not empty #}
<div class="sidebar-first"{{- section_attributes -}}>{# section_attributes and section_content are required to be present #}
  {{ section_content }} {# This contains the rendered content. #}
{% endsection %}
</div>

Internally in the template this would be a twig block:

function block__internal_[hash]($context, $blocks) {
  $context['section_content'] = render_var($context['sidebar_first']);
  if ($this->extension('drupal_core')->emptySection($context)) {
    return;
  }

  print '<div class="sidebar-first"';
  print $context['section_attributes'];
  print '>' . PHP_EOL;
  print $context['section_content'] . PHP_EOL;
  print '</div>' . PHP_EOL;
}

For a more advanced example we could allow:

{% section content.sidebar_first|strip_tags('html')|trim %}
{# This whole section will only be shown if the rendered variable filtered by the filters is not empty #}

{# section_attributes and section_content are required to be present #}
{{ section_content }} {# This contains the rendered content. #}
{% endsection %}

Remaining tasks

- Discuss technical solution

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Postponed: needs info

Version

11.0 πŸ”₯

Component

theme system

Created by

πŸ‡©πŸ‡ͺGermany Fabianx

Live updates comments and jobs are added and updated live.
  • stale-issue-cleanup

    To track issues in the developing policy for closing stale issues, [Policy, no patch] closing older issues

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΊπŸ‡ΈUnited States smustgrave

    Thank you for creating this issue to improve Drupal.

    We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

    Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

    Thanks!

Production build 0.71.5 2024