Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

baseline of sidenotes off #24

Closed
GoogleCodeExporter opened this issue Mar 12, 2015 · 8 comments
Closed

baseline of sidenotes off #24

GoogleCodeExporter opened this issue Mar 12, 2015 · 8 comments

Comments

@GoogleCodeExporter
Copy link

The side/foot notes are slightly off to the bottom. I added

    \leavevmode%

in l646 of tufte-common.sty of r86 to move the sidenotes up on the same 
baseline as the text. 
l624 has the command in \@footnotemark instead of \@footnotetext. 

This breaks the handout example layout, l159 of sample-handout.tex has a 
vertical offset.

Andy

Original issue reported on code.google.com by [email protected] on 13 Apr 2009 at 7:49

@GoogleCodeExporter
Copy link
Author

Oops, that was too quick. The first line of a sidenote does not move with the 
offset value any more. If the 
sidenote has more than 1 line the others do move up or down. strange...

Original comment by [email protected] on 13 Apr 2009 at 8:00

@GoogleCodeExporter
Copy link
Author

'Normal' Latex does the same.

\documentclass{article}
\usepackage {lipsum}

\begin{document}
  \lipsum[1]
  \marginpar{\vspace{0pt}\lipsum[1]}
  \lipsum[2]
\end{document}

misaligns the text in the margin.

\begin{document}
  \lipsum[1]
  \marginpar{\lipsum[1]}
  \lipsum[2]
\end{document}

does not.

Original comment by [email protected] on 13 Apr 2009 at 8:41

@GoogleCodeExporter
Copy link
Author

Try putting \hbox{} in front of the \vspace{#1} line.  It appears to fix the
alignment in the test document, but I haven't tested it further than that. 
(Currently working on putting captions alongside the figures/tables.)

Original comment by [email protected] on 13 Apr 2009 at 9:01

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

The hbox almost does it, but leads to a line skip. A \vspace*{-1\baselineskip}% 
after the font is changed to 
footnote font skips one line back up.

It is now (starting l644 in r86):

\renewcommand\@footnotetext[2][0pt]{%
  \marginpar{%
    \hbox{}% NEW
    \vspace*{#1}%
    \def\baselinestretch {\setspace@singlespace}%
    \ifthenelse{\boolean{@tufte@loadbidi}}{\if@rl@footnote\@rltrue\else\@rlfalse\fi}{}%
    \reset@font\footnotesize%
    \@tufte@margin@par% use parindent and parskip settings for marginal text
    \vspace*{-1\baselineskip}% NEW
    \noindent%
    \protected@edef\@currentlabel{%
       \csname p@footnote\endcsname\@thefnmark%
    }%
    \color@begingroup%
       \@makefntext{%
         \ignorespaces#2%
       }%
    \color@endgroup%
  }%
  \@tufte@reset@par% use parindent and parskip settings for body text
}%

Original comment by [email protected] on 13 Apr 2009 at 10:31

@GoogleCodeExporter
Copy link
Author

\marginnote should behave the same way I believe. 

l704 of tufte-common.sty r86 is now

  \marginpar{\hbox{}\vspace*{#1}\@tufte@marginfont\@tufte@justification\vspace*{-
1\baselineskip}\noindent #2}%

First, I was not sure if the offset given in lets say [-3\baselineskip] should 
skip margin lines or text lines 
(margin lines are smaller). But skipping in text lines makes alignment easier 
and, therefore, provides a cleaner 
look.

Original comment by [email protected] on 15 Apr 2009 at 4:10

@GoogleCodeExporter
Copy link
Author

I think I've caught all of them, but please double-check for me.  I just 
submitted
the code in r87 (alongside some major reworking of figure/table environments and
their captions).

Original comment by [email protected] on 15 Apr 2009 at 4:39

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

The sidenote alignment looks good to me now.

Original comment by [email protected] on 15 Apr 2009 at 9:21

@GoogleCodeExporter
Copy link
Author

Great!  I'll mark this as FIXED then.

Original comment by [email protected] on 15 Apr 2009 at 9:54

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant