In ✨ Optimize redirect chain retrieval in VariationCache Active , I noticed that we're currently rendering the umami_search block 3 times, because there's a repeated check for whether or not there are any blocks in those regions, first one of the two, then either, then actually rendering them.
As a demo theme, I think umami doesn't really need to worry about people removing those blocks?
If we for some reason do, we could store the rendered output into a variable and check and display that.
Active
11.0 🔥
Umami demo
It affects performance. It is often combined with the Needs profiling tag.
There has been similar code in Umami since it was committed to core, most recent change was 🐛 Umami page.tpl.php breaks block placeholders Needs review .
Also just found 🐛 Umami theme ignores placeholders and HTML replaced elements when checking for empty regions Needs work which looks similar.