search

edwardtufte / tufte-css

Style your webpage like Edward Tufte’s handouts.
https://edwardtufte.github.io/tufte-css/
MIT License
5.99k stars 459 forks source link

Align baseline of sidenote/marginnote with main text #154

Closed WesleyAC closed 4 years ago

WesleyAC commented 4 years ago

This adds 0.3rem of padding to the top of margin notes and sidenotes, causing the baseline of the first line of the note to be aligned with the baseline of the main body text.

As discussed in #134, this is the approach taken in Tufte's books.

Closes #134.

before: tufte_sidenote_before

after: tufte_sidenote_after

no impact on mobile, since the definition in the media query resets the padding already.

daveliepmann commented 4 years ago

I don't have time to look at this right now, but I should be able to take it for a spin within the next week or so. I appreciate that you took the time to include the screenshots, links, and explanation necessary to make this a smooth PR to review. Thanks.

daveliepmann commented 4 years ago

Looks great on my machine; strict improvement; no problems that I can see. Thanks!