On Thu, Sep 19, 2019 at 07:52:05AM +0900, Akira Yokosawa wrote: > From: Balbir Singh <bsingharora@xxxxxxxxx> > > Without these changes the PDF does not provide a browsable TOC > to choose for navigation, also add a bit of color to the links > to make them screen friendly. > > [akiyks: Split from patch [1]. > As 2c layouts are supposed to be for printing, enable coloring > of hyperlinks only in 1c layout. To do this, add a boolean > variable "colorlinks" and set it true for 1c layout. This would > make it easy to change the condition to enable "colorlinks". > > Select "DarkBlue" via xcolor's "svgnames" option as it looks > less distracting than the pure "blue". > > Redundant framing of links in bibliography is disabled when > "colorlinks" is effective. > > Also make the page numbers in TOC hyperlinks as well. > > [1]: https://www.spinics.net/lists/perfbook/msg02350.html > ] > > Signed-off-by: Balbir Singh <bsingharora@xxxxxxxxx> > Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx> > --- > Hi Paul and Balbir, > > This is an experimental patch. > Does the links in "DarkBlue" acceptable (in 1c layout) to you? > Any suggestion of color choice is welcome! Being as I am red-green colorblind, I must defer to the two of you on color choices. Looks just fine to me, but... ;-) Thanx, Paul > Thanks, Akira > -- > appendix/appendix.tex | 8 ++++++-- > perfbook.tex | 16 ++++++++++++++-- > 2 files changed, 20 insertions(+), 4 deletions(-) > > diff --git a/appendix/appendix.tex b/appendix/appendix.tex > index d246a7f7..7b0d5c76 100644 > --- a/appendix/appendix.tex > +++ b/appendix/appendix.tex > @@ -14,9 +14,13 @@ > }{ > \bibliographystyle{alphapf} % Use alpha style customized by urlbst with --inlinelinks option > } > -\hypersetup{pdfborder=0 0 1,urlbordercolor=0.4 1 1} > +\IfColorLinks{}{ > + \hypersetup{pdfborder=0 0 1,urlbordercolor=0.4 1 1} > +} > \bibliography{bib/RCU,bib/WFS,bib/hw,bib/os,bib/parallelsys,bib/patterns,bib/perfmeas,bib/refs,bib/syncrefs,bib/search,bib/swtools,bib/realtime,bib/TM,bib/standards,bib/OSS,bib/maze,bib/energy,bib/QC,bib/NP} > -\hypersetup{pdfborder=0 0 0} > +\IfColorLinks{}{ > + \hypersetup{pdfborder=0 0 0} > +} > \IfTwoColumn{ > \end{adjustwidth*}\twocolumn > }{ > diff --git a/perfbook.tex b/perfbook.tex > index 85696333..a0805831 100644 > --- a/perfbook.tex > +++ b/perfbook.tex > @@ -42,7 +42,7 @@ > \setlist[description]{style=unboxed} > %\usepackage{enumerate} > \usepackage{ifthen} > -\usepackage[table]{xcolor} > +\usepackage[table,svgnames]{xcolor} > \usepackage[shortcuts]{extdash} > \usepackage{changepage} > \usepackage{listings} > @@ -58,7 +58,7 @@ > \usepackage{tabularx} > \usepackage[hyphens]{url} > \usepackage{threeparttable} > -\usepackage[bookmarks=true,bookmarksnumbered=true,pdfborder={0 0 0}]{hyperref} > +\usepackage[bookmarks=true,bookmarksnumbered=true,pdfborder={0 0 0},linktoc=all]{hyperref} > \usepackage{footnotebackref} % to enable cross-ref of footnote > \usepackage[all]{hypcap} % for going to the top of figure and table > \usepackage{epigraph} > @@ -100,6 +100,18 @@ > \setboolean{nimbusavail}{false} > \newcommand{\IfNimbusAvail}[2]{\ifthenelse{\boolean{nimbusavail}}{#1}{#2}} > > +\newboolean{colorlinks} > +\setboolean{colorlinks}{false} > +\newcommand{\IfColorLinks}[2]{\ifthenelse{\boolean{colorlinks}}{#1}{#2}} > + > +\IfTwoColumn{}{ > +\setboolean{colorlinks}{true} > +} > + > +\IfColorLinks{ > +\hypersetup{colorlinks=true,allcolors=DarkBlue} > +}{} > + > \IfNimbusAvail{ > \usepackage{nimbusmononarrow} > }{} > -- > 2.17.1 > >