The HTML generated by system_preprocess_block (in system.module) when outputting menu blocks contains an attribute called 'classes' that contains 'block-menu'.

This was probably created by somebody accidentally writing out 'class' as 'classes' in the attributes array.

Wrong:

 <div id="block-footer"  class="block block-system contextual-region" role="navigation" classes="block-menu">

I will attach a patch after I get an issue number.

CommentFileSizeAuthor
#1 classes-2003388-1.patch494 bytesnicholaspaun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nicholaspaun’s picture

nicholaspaun CreditAttribution: nicholaspaun commented
Status: Active » Needs review
FileSize
classes-2003388-1.patch494 bytes
Sam Moore’s picture

Sam Moore
he/him
Location Sandy Hook, CT
CreditAttribution: Sam Moore commented
Status: Needs review » Reviewed & tested by the community

Patch in #1 works for me.

benjifisher’s picture

benjifisher
he/him or they/them
Primary language English
Location Boston area
CreditAttribution: benjifisher commented
Issue tags: +Quick fix

With a fresh install of D8, Minimal profile, the Tools menu on the home page looks like this before the patch:

<div id="block-tools" class="block block-system" classes="block-menu" role="navigation">
<h2>Tools</h2>
...
</div>

and after the patch it looks like this:

<div id="block-tools" class="block block-system block-menu" role="navigation">
<h2>Tools</h2>
...
</div>

+1 for RTBC

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

nice catch! Committed 7924d88 and pushed to 8.x. Thanks!

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