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

Comments

nicholaspaun’s picture

nicholaspaun commented
Status: Active » Needs review
StatusFileSize
new classes-2003388-1.patch494 bytes
sam moore’s picture

sam moore
he/him
Location Sandy Hook, CT
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
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 🇪🇺🌍
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.