search

pawroman / zola-theme-terminimal

A simple, minimal retro theme for Zola
https://pawroman.github.io/zola-theme-terminimal/
Other
346 stars 76 forks source link

Fix: link the top-left logo to `logo_home_link` when `logo_home_link` is set #39

Closed goingforbrooke closed 1 year ago

goingforbrooke commented 1 year ago

Issue

The bug is caused by the first missing .extra, which makes Tera assume that logo_home_link variable doesn't exist, so the if trigger for using it never fires. This means that the logo will always link to the default value of base_url.

The second missing .extra comes after the if fires. It's used to set the value of logo_link to logo_home_link.

Solution

This fix works by adding missing two missing .extra's when accessing the logo_home_link in the header template block.

Fixes #38.

Context

My current workaround is to extend the header block with the fix proposed here.

pawroman commented 1 year ago

Thanks for the fix!