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
Page breaks (burblebee) #4252
Page breaks (burblebee) #4252
Conversation
dc41e4d
to
7130f45
Compare
These changes should also be applied to master:
If we can't figure out what is causing the spurious blank line, we'll also want to merge this which removes the
|
Please remove the "partially addresses" note from the individual commit messages. We want that on the pull request, not in the commit messages. |
7130f45
to
935d50c
Compare
done. |
[temp.dep.type]p9 has a mid-sentence note. Consider adding a page break before [cpp.replace.general]p10 Please see if you can move the declarations before [new.delete.single]p10 all onto the same page without reflowing that page -- it looks like there might be enough white space at the bottom for that. If it doesn't fit, try putting two of the declarations on each page instead. [string.view.synop] would probably benefit from a page break before. The There's a spurious line break in table [tab:container.seq.opt] in the description of Otherwise, looks good. Thanks! |
935d50c
to
48ed0ba
Compare
I fixed all the issues you raised above.
I can't explain this one - looks like a bug in Latex, triggered when a sentence ends at the column boundary and is followed by |
48ed0ba
to
23e7280
Compare
23e7280
to
7262b80
Compare
Thanks, looks great! (I can't actually approve because I created this PR, but... approved!) |
Partially addresses #4228