grantmcdermott / quarto-revealjs-clean

A minimalist and elegant presentation theme for Quarto Reveal.js
https://grantmcdermott.com/quarto-revealjs-clean-demo/template.html
MIT License
230 stars 37 forks source link

Multiline titles are written over each other #2

Closed ryancbriggs closed 1 year ago

ryancbriggs commented 1 year ago

Multiline titles are written over each other. This is easily fixed by changing line-height in .title from 0 to 1. (line 47 clean.scss)

grantmcdermott commented 1 year ago

Thanks Ryan. Busy day for me this side, but will test and push later.

grantmcdermott commented 1 year ago

I implemented a slightly different solution, since I want minimal space between the title and subtitle on the title page itself... but should be working now. Please give it a go when you have a sec.

ryancbriggs commented 1 year ago

Your changes give me this: vs. when I do this: .title { color: $body-color; font-size: 1.6em; font-weight: 350; line-height: 1; }

I get this: 

On Jul 19, 2023, at 12:32 PM, Grant McDermott @.***> wrote:

I implemented a slightly different solution, since I want minimal space between the title and subtitle on the title page itself... but should be working now. Please give it a go when you have a sec.

— Reply to this email directly, view it on GitHub https://github.com/grantmcdermott/quarto-revealjs-clean/issues/2#issuecomment-1641752479, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJOS64X5B33JWCBLFOHOIYTXQ6SUBANCNFSM6AAAAAAZ7D6SVQ. You are receiving this because you authored the thread.

grantmcdermott commented 1 year ago

Sorry, missed this on vacation. But I can’t actually see any images in your reply. Can you try to upload manually (or just post the quarto slide content that you are knitting so I can try on my end)?