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

enable pdftex extensions with \usepackage{microtype} #584

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Shark64
Copy link

@Shark64 Shark64 commented Dec 13, 2013

I've tried to enable microtype and seems to work well (fewer overfull hboxes).
The only problem was hyphenation of 2 section titles, but disabling tracking on those 2 pages seems to fix it.

@mikeshulman
Copy link
Contributor

This is a good idea, but not being very familiar with microtype myself, I'm a little uncomfortable merging it without paging through the entire document to make sure it looks better, and I don't have time to do that myself right now.

@Shark64
Copy link
Author

Shark64 commented Dec 13, 2013

Well, to be fair, i didn't look carefully to the entire book either. In my experience microtype usually works well, it's disabled by default only to make PdfTeX's output 100% equivalent with that of TeX, but the changes are quite subtles so i agree, it's not an high priority bug.

@mikeshulman
Copy link
Contributor

I suppose we could just rely on the magic of github to hope that our readers eventually report any other formatting problems that this introduces. Anyone else have an opinion?

@andrejbauer
Copy link
Member

This is a good idea, but I think we do need to page through the entire books (all of them) to catch possible problems. I kindly ask volunteers to try out this pull request and report on how well it works. It's best to take one version of the book and compare the old vs. new. I am mostly worried about the ustrade size because that one gets printed.

Shark64 added 2 commits December 17, 2013 13:03
enable pdftex extensions with \usepackage{microtype}
@mikeshulman mikeshulman mentioned this pull request Dec 31, 2013
@peterlefanulumsdaine
Copy link
Contributor

I’ve asked a question about this on tex.stackexchange — asking what we should be checking for after adding microtype.

In any case: it seems to me more trouble than it’s worth to add microtype to the current edition, since it would require re-proofing the whole book. But for the second edition (if that happens), the book will need to be re-proofed anyway, so there’s presumably no reason not to add it then. So I guess we should keep this PR around, but labelled as “for 2nd ed”?

@mikeshulman
Copy link
Contributor

Makes sense to me.

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

Successfully merging this pull request may close these issues.

4 participants