Re: [libvirt] [PATCH] docs: add a permalink to html headers
On 08.12.2013 23:57, Dan Kenigsberg wrote:
Note to reviewers: this patch is too naive and has devastating effect
of
headers that lack a named anchor element. Still, I'd love to hear your
opinion about the idea.
On Sun, Dec 08, 2013 at 02:05:46PM +0000, Dan Kenigsberg wrote:
> Quite often, I need to cite URLs like
> http://libvirt.org/formatnetwork.html#elementQoS
> but it is annoying to copy them from the table of contents or the html
> source.
>
> This patch borrows from the Python documentation in order to make it
> easier to cite headers on libvirt's oneline documentation.
> ---
> docs/libvirt.css | 13 +++++++++++++
> docs/page.xsl | 7 +++++++
> 2 files changed, 20 insertions(+)
I like the idea - I often find myself in the same situation, but if we decide to merge
this, we need to fix some issues:
Validating internals/command.html
./html/libvirt-libvirt.html:6: element a: validity error : No declaration for attribute
xmlns:html of element a
ne">blah</a></h2><h3><a
xmlns:html="http://www.w3.org/1999/xhtml" href="#macros"
^
(line repeats over and over)
Moreover, going with 'ΒΆ' turns out to be not cool:
Generating logging.html.tmp
../docs/page.xsl:192: parser error : Input is not proper UTF-8, indicate encoding !
Bytes: 0xB6 0x3C 0x2F 0x61
<a class="headerlink" href="#{html:a/@name}" title="Permalink
to this headline">
Michal