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

Display sidenote numbers outside the sidenote box. #9

Closed
wants to merge 1 commit into from

Conversation

benjamn
Copy link

@benjamn benjamn commented Aug 5, 2015

This way the sidenote text will not be indented by the number.

I think the correct term for this typographical style is "hanging punctuation" (https://en.wikipedia.org/wiki/Hanging_punctuation).

This way the sidenote text will not be indented by the number.

I think the correct term for this typographical style is "hanging
punctuation" (https://en.wikipedia.org/wiki/Hanging_punctuation).
@benjamn
Copy link
Author

benjamn commented Aug 5, 2015

Hmm, the numbers appear a little higher in Firefox. Not sure why.

@benjamn
Copy link
Author

benjamn commented Aug 5, 2015

I should also mention: I'm not sure this is what Tufte would want. This pull request is more of a proposal than a normative recommendation or bug-fix.

@daveliepmann
Copy link
Contributor

Hmm. This is quite an interesting idea. At first flush I was in favor, but A) the FF bug should be resolved and B) I just checked the Visual Display of Quantitative Info (the only book Tufte has that Amazon allows me to virtually 'look inside') and Tufte does indent sidenotes by number.

I'd be interested in some comments both pro and con.

daveliepmann added a commit that referenced this pull request Aug 8, 2015
@daveliepmann
Copy link
Contributor

Hey, I'm sorry, I tried to merge on the command line (because of a conflict) but somehow it didn't get tagged as you. I'm using your code, though, so all credit goes to you, and thanks!

@benjamn
Copy link
Author

benjamn commented Aug 8, 2015

All good!

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

Successfully merging this pull request may close these issues.

2 participants