block-content-add-list.html.twig does...

  <dl class="node-type-list">
    {% for type in types %}
      <dt>{{ type.link }}</dt>
      <dd>{{ type.description }}</dd>
    {% endfor %}
  </dl>

It's not a node-type-list.

Let's just remove the unnecessary class.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Bug because content blocks are not nodes.
Unfrozen changes Unfrozen because it only changes a template
Disruption Very minor disruption is a theme relies on the block content type listing having this class. Core does not use this class.
Files: 
CommentFileSizeAuthor
d8.block-content-type-template.patch520 bytesalexpott
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 84,841 pass(es). View

Comments

davidhernandez’s picture

Just confirming that I see no CSS or JS attached to this class.

Cottser’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix

Thanks @alexpott!

For a little bit of history, take a peek at #1871772-108: Convert custom blocks to content entities which calls out this "wrong" class name, but no follow-up happened.

  • catch committed d8d284d on 8.0.x
    Issue #2412373 by alexpott: Remove node-type-list from block-content-add...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.