The output of these two functions is identical, let's please consolodate.

<div class="admin clearfix">
  {{ system_compact_link }}
  {% for class, content in container %}
    <div class="{{ class }} clearfix">
      {{ content }}
    </div>
  {% endfor %}
</div>

Comments

steveoliver’s picture

Assigned: Unassigned » steveoliver
jenlampton’s picture

Issue tags: +Template consolidation

tagging

joelpittet’s picture

Assigned: steveoliver » Unassigned
Issue summary: View changes

What should the consolidated template be called?

admin-page.html.twig


{{ system_compact_link }} {% for container in containers %}
{% for block in container.blocks %} {{ block }} {% endfor %}
{% endfor %}

system-admin-index.html.twig


{{ system_compact_link }} {% for position, blocks in containers %}
{% for block in blocks %} {{ block }} {% endfor %}
{% endfor %}
mitokens’s picture

andypost’s picture

Title: Consolidate use of theme_admin_page() and theme_system_admin_index() » Consolidate use of admin-page.html.twig and system-admin-index.html.twig
Version: 8.0.x-dev » 9.x-dev
Status: Active » Postponed

this is a part of stable theme

catch’s picture

Version: 9.x-dev » 8.2.x-dev

We can do the consolidation of usages + deprecation in a minor release, just can't do the actual removal.

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.0-beta1 was released on August 3, 2016, which means new developments and disruptive changes should now be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.0-alpha1 will be released the week of January 30, 2017, which means new developments and disruptive changes should now be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.