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
[intro.abstract] Move \indextext under \pnum #2088
Conversation
This issue probably only affects the HTML version generated by cxxdraft-htmlgen, so in fairness this should probably be considered a cxxdraft-htmlgen deficiency. Having said that, fixing it by changing the LaTeX means less work for cxxdraft-htmlgen, so I'm all for it. :P |
Completely disagree. |
As far as LaTeX is concerned, the index entry is just a location you can jump to. With an index entry right before a pnum, the jump location coincides with the start of the paragraph, so that if you jump to the index entry, you correctly arrive at the start of the paragraph. As far as I know this is ordinary LaTeX behavior. On what do you base your "the current location is the last defined paragraph" interpretation? |
Yep. And the location where the index entry was defined — was the end of the paragraph. Not the beginning.
How an index entry should coincide with the start of a paragraph which haven't even started yet?
Have you ever used "ordinary" LaTeX? For example, Anyway, just read https://github.com/cplusplus/draft/wiki/Specification-Style-Guidelines#source-code-formatting |
I'm not sure why you think there is trouble in having a (zero-width) location coincide with the start of the thing that follows it. In the string "ABC", the location between B and C coincides with the start of C. No? As for \label, that's a different command. It doesn't tell us anything about index entries. Anyway, I'm happy to see that the guidelines recommend putting the index entry after the pnum. I suspect it's just for human source readability since LaTeX doesn't care, but it's great for cxxdraft-htmlgen! :) |
A closely related question that the style guidelines do not address, is whether \indextext entries for list items should be placed after the \item as well. For eel.is/c++draft I currently use a patch that moves some such index entries that currently occur before \item, for much the same reason as described in the description of this pull request. So if the style guidelines were updated to say that index entries for list items are also to be placed after the \item command, then that would be great for me. |
I wouldn't mind such a style rule, though I'm not sure if anyone would spend significant effort on modifying existing uses. |
The next paragraph starts after the blank line, not at the |
I think this is not a style rule, because it affects which paragraph an index entry points to.
No. The blank line ends the current paragraph: The |
Yes, you're right. I think what I meant was that I'd only have such a rule apply to when the index entry is in fact meant to appertain to the following paragraph.
Wonderful! |
Currently index entries point to the end of the previous paragraph, see pic.