[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] ox-html: add option to restore old src block behaviour?
From: |
Jens Lechtenboerger |
Subject: |
Re: [O] [PATCH] ox-html: add option to restore old src block behaviour? |
Date: |
Tue, 08 Oct 2019 20:40:22 +0200 |
On 2019-10-08, at 11:31, Nicolas Goaziou wrote:
> Jens Lechtenboerger <address@hidden> writes:
>
>> The attached patch adds a new variable org-html-wrap-src-lines to
>> control whether code tags should be added or not.
>
> Thank you.
>
> However, the patch is not right. Exporters do not use defcustoms
> directly. Instead, you register them within :options-alist in the the
> back-end definition, e.g., under :html-wrap-src-lines and then call
> (plist-get info :html-wrap-src-lines) in the function. This allows to
> control these options during publishing.
Indeed, sorry. Many thanks for the detailed feedback!
I had to change the calling function org-html-format-code and add an
argument to org-html-do-format-code as info is not available in the
latter function.
> Also, this would need an entry in the manual, if only in the "options
> for the exporters" subsection.
Done.
>> I’m not sure whether :package-version 9.3 is correct.
>
> It is correct. You can also use :safe t.
Added.
>> Also, I set the value to t, which does not change the current
>> functionality. However, for backwards compatibility (up to version
>> 9.2.6), a value of nil would be preferable. Any thoughts?
>
> I agree with the nil default value.
Done.
>> +(defcustom org-html-wrap-src-lines t
>> + "If t, wrap individual lines of source blocks in \"code\" elements.
>
> When non-nil, wrap...
Done.
Best wishes
Jens
0001-ox-html-Control-wrapping-of-source-lines.patch
Description: Text Data