On 2016/04/24 12:50, Akira Yokosawa wrote:
> On 2016/04/24 10:08, Akira Yokosawa wrote:
>> >From 6ecb00b37832372d18cdc7ff5eaea9a53eab3cb9 Mon Sep 17 00:00:00 2001
>> From: Akira Yokosawa <akiysw@xxxxxxxxx>
>> Date: Sun, 24 Apr 2016 08:19:20 +0900
>> Subject: [PATCH v3 1/3] qqz: Improve accuracy of cross-links
>>
>> In commit 33b93f8258f5 ("qqz: Cross-link questions and answers"),
>> cross-links of quick quizzes and their answers were implemented.
>> However it uses the \stepcounter{} command, and direct references
>> of 'quickquizctr' counter is not output to the .aux file.
>> The symptom is when you click on a hyperlink in the resulting PDF,
>> you jump to a random nearby label such as that of Figure, Table,
>> or Section, etc. which happens just before the \label{} using
>> 'quickquizctr'.
>>
>> By using \refstepcounter{} instead and using 2 counters
>> 'quickquizctr' and 'quickquizctrC' ('C' stands for 'Chapter'),
>> each of which is automatically reset at a new chapter and
>> a new section respectively, and defining \theHNum at the head
>> of qqz.sty, then redefining it within the \QuickQuizAnsers
>> command, matching QQ.n.m's and QQA.n.m's are output to the .aux
>> file correctly.
>>
>> By applying this change, resulting hyperlinks become to jump to
>> places almost exactly where you'd expect.
>>
>> This change may have the effect of increasing the required number
>> of 'pdflatex' iterations to get a final result. The following
>> commit will take care of it.
>>
>> This commit also updates the copyright notices in the header,
>> including the name of Paolo Bonzini who authored the commit
>> mentioned above.
>>
>> Signed-off-by: Akira Yokosawa <akiysw@xxxxxxxxx>
>
> Hi, Paul.
>
> I noticed there really is a change in the behavior of hyperref's of
> Chapters. They used to jump to the top of the pages, but after
> this patch is applied, they jump to the bottom of the Chapter titles.
> So you need to scroll a little to see the title of the Chapters.
>
> I think this is a regression caused by this patch.
>
> If you do think so, please keep from pushing this series out.
>
> Or the benefit of the patch is big enough compared to the change
> of the behavior?
>
> Regards, Akira
>
Hi,
Seems like it is not easy (at least for me) to find out what
is wrong in this patch.
So there are a few options I can think of you might want to do
(assuming you'd like to push the refactoring of runlatex.sh):
1) Apply the series as a whole, then revert [1/3], and push out.
2) Me resubmit another patch series which just contains the refactoring.
3) Apply the series as a whole, and push it as is in the hope
of someone familiar with LaTeX and hyperref might appear.
Thoughts?
Regards, Akira
--
To unsubscribe from this list: send the line "unsubscribe perfbook" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
