fix(analytics): Matomo analytics

[ldez]

I forgot the with and to fill the TODO.

[netlify]

✅ Deploy Preview for hugo-hextra ready!

Name	Link
🔨 Latest commit	acd18d0
🔍 Latest deploy log	https://app.netlify.com/projects/hugo-hextra/deploys/68b80ff9e86efd000829b08e
😎 Deploy Preview	https://deploy-preview-817--hugo-hextra.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[imfing]

this is also checked at

hextra/layouts/_partials/components/analytics/analytics.html

Line 15 in cc5884d

{{- if .Site.Params.analytics.matomo -}}

I guess we don't need both?

[ldez]

It's 2 different things:

• The if is purely conditional and doesn't change the scope of .
• The with is conditional, but it also changes the scope of .

IMHO, it's better to have a larger scope at the global partial level (because it's easier to override), and inside the partial have a restricted scope for readability and maintenance.

[ldez]

🤔 If someone wants to override (to a specific analytics), he can either override layouts/_partials/components/analytics/analytics.html or add an element inside layouts/_partials/custom/head-end.html.

So I think I will use with.

[imfing]

It's 2 different things:

• The if is purely conditional and doesn't change the scope of .
• The with is conditional, but it also changes the scope of .

IMHO, it's better to have a larger scope at the global partial level (because it's easier to override), and inside the partial have a restricted scope for readability and maintenance.

I see, that makes more sense, I think I’d actually prefer giving the partial a broader scope now 🤔