Re: [PATCH] qqz: Add another hyperlink to black box in QQA

["Paul E. McKenney" <paulmck@linux.vnet.ibm.com>]

On Mon, Jul 18, 2016 at 07:54:27AM +0900, Akira Yokosawa wrote:
> On 2016/07/17 14:59:00 -0700, Paul E. McKenney wrote:
> >On Sun, Jul 17, 2016 at 02:50:04PM +0900, Akira Yokosawa wrote:
> >>From 2a74b30f0a1d036e45c6aa5ef431034c23f790db Mon Sep 17 00:00:00 2001
> >>From: Akira Yokosawa <akiyks@gmail.com>
> >>Date: Tue, 12 Jul 2016 20:28:08 +0900
> >>Subject: [PATCH] qqz: Add another hyperlink to black box in QQA
> >>
> >>This commit adds another cross-link to the black boxes in QQA.
> >>I think this change would meet readers' expectation.
> >>
> >>Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> >
> >I agree!  For reasons that are not clear to me, I had to hand-apply
> >this patch.  Might have been a white-space issue, which I did not
> >think to check for...
> >
> >Please take a look at the patch below to make sure that I did it
> >correctly.
> 
> Yes, you did it right. I'm wondering what was wrong on my side.

My guess is whitespace.  Or maybe the email got mangled along the way.

> >Also, this suggests a possibility:  In the quick-quiz answer, suppose
> >that the heading and the black box link (as they do now) to the beginning
> >of the quick-quiz question, but that the white box links to the end
> >(or just after the end) of the quick-quiz question.  That way, the
> >reader could choose the return point.
> >
> >Would that make sense?
> 
> Well, yes if readers expect the behavior.
> But for long answers, there are cases where you can choose only the white
> boxes in your view. I'm hesitating because of your reaction to the first
> attempt  the other day.

All good points!

> Anyway, I'm going to submit a patch for you to try.
> 
> Also we'd also need to update the explanation in the answer of Quick Quiz 1.3
> if we actually do the change.

Looking forward to it!

						Thanx, Paul

> >------------------------------------------------------------------------
> >
> >commit 8afdb9c1a55c7cf2300e6812cff58dd0fda4d585
> >Author: Akira Yokosawa <akiyks@gmail.com>
> >Date:   Sun Jul 17 14:54:03 2016 -0700
> >
> >    qqz: Add another hyperlink to black box in QQA
> >
> >    This commit adds a hyperlink from the black box in the quick-quiz
> >    answer to the beginning of the corresponding quick-quiz question.
> >
> >    Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> >    Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> >
> >diff --git a/qqz.sty b/qqz.sty
> >index 12dc85e30116..0c97df92cb50 100644
> >--- a/qqz.sty
> >+++ b/qqz.sty
> >@@ -78,7 +78,8 @@
> > 	\noindent
> > 	\QuickQHeading{QQA}{QQ}{\QuickQuizAnswerChapter.\thequickquizctrC} #1 ~ \\ }
> >
> >-\newcommand{\QuickA}[1]{\rule{7pt}{7pt}
> >+\newcommand{\QuickA}[1]{%
> >+	\hyperref[QQ.\QuickQuizAnswerChapter.\thequickquizctrC]{\rule{7pt}{7pt}}
> > 	\par\vspace{0.8\baselineskip}
> > 	\noindent\textbf{Answer:} \\ }
> >
> >
> >
>