-
-
Notifications
You must be signed in to change notification settings - Fork 278
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
Footmisc bug 19 #1701
base: develop
Are you sure you want to change the base?
Footmisc bug 19 #1701
Conversation
If you happen to look at the last commit @CastleStar14654 please ignore it, I must have been already asleep when I wrote that nonsense. But now it is too late and I wil fix/revert it tomorrow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, but still possibly merge the baseline attach sockets
previous behavior, they are by default both \plug{off}. One probably | ||
wants to set them both to \plug{on} or both to \plug{off}, but for | ||
maximum flexibility the two can be individually changed. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I (think) I'd lean towards combining these.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it sure is possible to merge the two and in fact I'm also leaning towards that since everything else seems artifical, but without feedback I thought I stay with both.
Internal housekeeping
Status of pull request
Checklist of required changes before merge will be approved
\changes
entries in source includedchanges.txt
updatedltnewsX.tex
(and/orlatexchanges.tex
) updatedThis fixes a bug reported for footmisc
FrankMittelbach/fmitex-footmisc#19
but is also present in the kernel code