From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Goaziou Subject: Re: [PATCH] ox-html.el ox.el: added list of figures support in html backend Date: Thu, 12 Nov 2015 23:46:23 +0100 Message-ID: <87k2pmu98g.fsf@nicolasgoaziou.fr> References: <87611btppi.fsf@snow.nl> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:57078) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zx0bi-0007hW-0C for emacs-orgmode@gnu.org; Thu, 12 Nov 2015 17:44:42 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zx0bd-00055s-Uk for emacs-orgmode@gnu.org; Thu, 12 Nov 2015 17:44:41 -0500 Received: from relay3-d.mail.gandi.net ([217.70.183.195]:59647) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zx0bd-00055n-PH for emacs-orgmode@gnu.org; Thu, 12 Nov 2015 17:44:37 -0500 In-Reply-To: <87611btppi.fsf@snow.nl> (Joost Helberg's message of "Mon, 09 Nov 2015 11:46:49 +0100") List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: Joost Helberg Cc: emacs-orgmode Hello, Joost Helberg writes: > this patch adds the list of figures to the html export backend. It also > includes a translation entry-point for the label "List of Figures" with > already one translation in: dutch. Thank you. > From fab9105d04e5cb9f923c17d322d84a25527ec27a Mon Sep 17 00:00:00 2001 > From: Joost Helberg > Date: Sun, 8 Nov 2015 14:34:11 +0100 > Subject: [PATCH] ox-html.el: added list of figures support in html > backend It should be ox-html: Add list of figures > * lisp/ox-html.el (org-html-list-of-figures): new, similar to > org-html-list-of-tables but with tables replaced by figures. This is, indeed, almost the same as `org-html-list-of-tables' and `org-html-list-of-listings'. I suggest to refactor the 3 functions to use a generic one. Also please use a recent version of `org-html-list-of-tables' since yours contain deprecated code (e.g., `org-export-solidify-link-text'). > (org-html-list-of-tables): added mapping keyword "figures" > to new function org-html-list-of-figures. You can remove this. > ox.el: added label for "list of figures" Ditto. > * lisp/ox.el (org-export-dictionary): Added label for List of > Figures with dutch translation You need to add TINYCHANGE at the end of the commit message, if you haven't signed FSF papers yet. > +(defun org-html-list-of-figures (info) > + "Build a list of figures. > +INFO is a plist used as a communication channel. Return the list > +of figures as a string, or nil if it is empty." > + (let ((lol-entries (org-export-collect-figures info nil))) (org-export-collect-figures info nil) is rather sloppy. It collects all paragraphs with a caption. Is it a proper definition for "figures"? Regards, -- Nicolas Goaziou