This is probably the last on my immediate wishlist of theme cleanup tasks.

I want to revisit the menu-related theming hooks, but my "what the heck is the use-case for this" argument is dead because I was actually to happy to have sitemap-specific preprocess functions for the D7 version earlier this week.

CommentFileSizeAuthor
#5 sitemap_2613790-4.patch12.06 KBakalata
#2 sitemap_2613790-2.patch0 bytesakalata
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

akalata created an issue. See original summary.

akalata’s picture

akalata CreditAttribution: akalata commented
Status: Active » Needs review
FileSize
sitemap_2613790-2.patch0 bytes

Status: Needs review » Needs work

The last submitted patch, 2: sitemap_2613790-2.patch, failed testing.

The last submitted patch, 2: sitemap_2613790-2.patch, failed testing.

akalata’s picture

akalata CreditAttribution: akalata commented
FileSize
sitemap_2613790-4.patch12.06 KB

And that's what happens when I don't sanity-check the patch. Also why I like testbot to do the heavy lifting before a commit.

akalata’s picture

akalata CreditAttribution: akalata commented
Status: Needs work » Needs review

The last submitted patch, 2: sitemap_2613790-2.patch, failed testing.

The last submitted patch, 2: sitemap_2613790-2.patch, failed testing.

akalata’s picture

akalata CreditAttribution: akalata commented
Status: Needs review » Fixed

  • akalata committed 0e3291c on 8.x-1.x 
    Issue #2613790 by akalata: Combine sitemap.html.twig and sitemap-box....

Status: Fixed » Closed (fixed)

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