Repository navigation
PEP 366, 395, 413, 426, 430, 440, 531, 535, 3150: Fix footnotes - #2794
Conversation
…emove redundant emacs metadata
…nt emacs metadata
CAM-Gerlach
left a comment
There was a problem hiding this comment.
I checked through the source changes and spot checked the rendered versions and they LGTM overall, but the bulleted inline-linked items did look kind of strange alongside the numbered footnotes with raw links. Its not perfect (we probably should also tweak the vertical margins of things at some point, not just here but a few other places could use them too), but converting the footnotes that are just links with a title to actual inline links improves this a lot and makes them more consistent and coherent, as well as being cleaner and taking up less space overall. Do you think you could do that at least for the ones where you added bulleted references?
vs.
|
Good idea, I'll update them. |
d42816b to
63b9008
Compare
CAM-Gerlach
left a comment
There was a problem hiding this comment.
Thanks, looks pretty good—one thing though, the interleaved bullets and footnotes are a little confusing to me, as it looks like the bulleted items are sub-items of the footnotes. Could you maybe just move the non-numbered footnote bulletted items to the end after the footnotes?
|
Done! |
|
Thanks for the reviews! |


Fix these warnings:
Details
PEP 366:
PEP 395:
PEP 413:
PEP 426:
PEP 430:
PEP 440:
PEP 531:
PEP 535:
PEP 3150:
Previews