maieul / latexhumain

LaTeX et sciences humaines
48 stars 11 forks source link

Warnings du package hyperref #14

Closed AKielhorn closed 9 years ago

AKielhorn commented 11 years ago

Bonjour!

I hope you don't mind if i write in english, my french is rather limited.

There are several cases of

Package hyperref Warning: Token not allowed in a PDF string (Unicode): (hyperref) removing `\delimiter' on input line 79.

or similar.

This can be corrected with \texorpdfstring{}{} as shown in the examples below. The first is just to keep TeX quiet, the second creates incorrect PDF bookmarks. (There are more cases, I just picked these two.)

Should I fork and submit a pull request or is this issue sufficient?

citation.tex: \section[Appel du package]{Appel du package \package{biblatex}}

\section[Appel du package]{\texorpdfstring{Appel du package \package{biblatex}}{Appel du package biblatex}}

\subsection{Les arguments \arg{prenote} et \arg{postnote}}

\subsection{\texorpdfstring{Les arguments \arg{prenote} et \arg{postnote}}{Les arguments prenote et postnote}}

Axel

maieul commented 11 years ago

thanks for your ticket. I was not looking for these warning, because I didn't see the problem.

I think the \texorpdfstring should be made inside the \arg, \package etc command, and not by duplicating the text.

If you want to do it yourself, I will be very grateful, but if don't want, I will do it one day...

AKielhorn commented 11 years ago

If you want to do it yourself, I will be very grateful, but if don't want, I will do it one day...

I have forked the repository and will look into it over the weekend.

Axel

maieul commented 9 years ago

hi, some news?

maieul commented 9 years ago

I close this issue. Reopen if needed.