crazko / romanvesely.com

:man: personal site
https://romanvesely.com
1 stars 2 forks source link

Upgrade Statie to 5.3 #52

Closed TomasVotruba closed 5 years ago

crazko commented 5 years ago

Thanks man, I just need to test the whole site. I think anchors in the post headings have been removed.

markdown_headline_anchors => post_headline_linker_enabled

TomasVotruba commented 5 years ago

Yes. Add it on merge after you test the site, if that's not an issue

crazko commented 5 years ago

hmm, it is not working properly, there are parts of the layout that are missing.

TomasVotruba commented 5 years ago

What exactly?

crazko commented 5 years ago

Here https://github.com/crazko/romanvesely.com/blob/master/site/_layouts/post.latte

{block content} is not used, but the content of the post is here https://github.com/crazko/romanvesely.com/blob/master/site/_layouts/default.latte#L59 directly

But {block meta} and {block scripts} are both correctly in the output.

crazko commented 5 years ago

I've just tried to rename {block content} to {block foo} and it's working. So I assume that $post['content'] and {block content} are fighting together.

TomasVotruba commented 5 years ago

Ah, that's the same issues as in statie.org. The post should be standalone template

crazko commented 5 years ago

Yeah, well, in that case I'll stick with the older version.

TomasVotruba commented 5 years ago

Do you want to go in depth yourself as in statie.org, which will take rather big amount of time, or can I create post.latte with includes as in statie.org?

crazko commented 5 years ago

Thanks, but I'd rather keep layout inheritance as it is now, so I'm fine with the older version of Statie now.

TomasVotruba commented 5 years ago

Understood :+1: