search

gicrisf / zhuia

An elegant but still playful theme for Zola.
Other
17 stars 10 forks source link

Add math rendering with Katex #1

Closed rlee287 closed 2 years ago

netlify[bot] commented 2 years ago

Deploy Preview for zhuia ready!

Name Link
Latest commit 3a8d1b964360a47b9068027e0216d0b22b8abb1a
Latest deploy log https://app.netlify.com/sites/zhuia/deploys/62abfbe274ad140008cf7ecc
Deploy Preview https://deploy-preview-1--zhuia.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

gicrisf commented 2 years ago

Hi! Thank you for your PR! I'm sorry for the late response, but I'm drowning with the stuff to do in college, so I didn't have time. At a first glance, it looks like you completed the integration with LaTeX exactly like I planned to: with opt-in/out in config file and a per file basis. Am I right? The file you removed in the first commit was a macro of yours?

rlee287 commented 2 years ago

The file that I removed was named "2", which exists in the main branch at templates/macros/2 and which looked like a file accidentally left by someone else by a mistyped shell redirection. The theme worked for me after I removed it, but it should probably get a more descriptive name if it is supposed to do something.

gicrisf commented 2 years ago

The file that I removed was named "2", which exists in the main branch at templates/macros/2 and which looked like a file accidentally left by someone else by a mistyped shell redirection. The theme worked for me after I removed it, but it should probably get a more descriptive name if it is supposed to do something.

Yes! You're absolutely right. I checked everything just now, sorry for the long wait. On the other hand, your macro is just fine, so I'm going to merge. Thanks again