Themeing messed up after D9 upgrade

Created on 16 December 2021, over 3 years ago
Updated 13 August 2024, 8 months ago

Is it by purpose that those parts did fell out of sitemap.theme.inc?

/**
 * Preprocesses the rendered tree for theme_sitemap_menu_tree().
 *
 * This is a clone of the core template_preprocess_menu_tree() function
 * with the exception of the sitemap specific class name used in the
 * UL that also allow themers to override the function only
 * for the sitemap page.
 */
function template_preprocess_sitemap_menu_tree(&$variables) {
  $variables['tree'] = $variables['tree']['#children'];
}

And

function theme_sitemap_menu_tree(array $variables) {
  return '<ul class="sitemap-menu">' . $variables['tree'] . '</ul>';
}

we now have the following classes in the sitemap menue tree

<div>
  <ul class="nav">
                                          <li class="nav-item  dropdown">
                      <a href="/test" class="nav-link dropdown-toggle" data-toggle="dropdown" data-drupal-link-system-path="node/17">test</a>
                          <div class="dropdown-menu">
              <a href="/test/untertest" class="dropdown-item" data-drupal-link-system-path="node/18">untertest</a>
  </div>
                              </li>
            </ul>
  </div>

which, at firsthand comes out as a complete mess.
Don't know wether this nav-item class is used somewhere else on the site and styling it breaks it in other places.

A mention of that in the (not jet updated) Readme.txt would be great.

💬 Support request
Status

Closed: outdated

Component

Miscellaneous

Created by

🇩🇪Germany macdev_drupal Wiesbaden

Live updates comments and jobs are added and updated live.
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.

Production build 0.71.5 2024