emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] [PATCH] New LaTeX exporter, switch for hypersetup


From: Nicolas Goaziou
Subject: Re: [O] [PATCH] New LaTeX exporter, switch for hypersetup
Date: Thu, 10 Jan 2013 22:22:24 +0100

Hello,

address@hidden (Thomas S. Dye) writes:

Thanks for your patch. A few comments below.

> I don't think I fully understand how :options-alist works.  With the
> attached patch I was expecting to be able to use #+LATEX_HYPER: nil, but
> this didn't work for me.

Keywords values are either a string or a list of strings. In this case,
#+LATEX_HYPER: nil means the value is "nil", not nil.

You still can test it with `org-not-nil' function.

>    :options-alist ((:date "DATE" nil org-e-latex-date-format t)
>                 (:latex-class "LATEX_CLASS" nil org-e-latex-default-class t)
>                 (:latex-class-options "LATEX_CLASS_OPTIONS" nil nil t)
> -               (:latex-header-extra "LATEX_HEADER" nil nil newline)))
> +               (:latex-header-extra "LATEX_HEADER" nil nil newline)
> +               (:with-hyper "LATEX_HYPER" "texht" t t)

I suggest to use :latex-hyperref-p or :latex-with-hyperref instead
of :with-hyper, since this is back-end specific.

Moreover, IMO, it doesn't make sense to provide a way to set it both
through the #+OPTIONS: line and with a "LATEX_HYPER" keyword. You should
choose your side.

Also, you may want to use a global defcustom for the default value,
instead of setting it to t, i.e. `org-e-latex-with-hyperref'.

> +               ))

Please do not leave parenthesis on a line of their own.


Regards,

-- 
Nicolas Goaziou



reply via email to

[Prev in Thread] Current Thread [Next in Thread]