openSUSE / suse-xsl

DocBook XSL Stylesheets for SUSE branding
Other
11 stars 10 forks source link

glosslist broken in HTML #432

Open tomschr opened 3 years ago

tomschr commented 3 years ago

Problem Description

The current stylesheets creates broken HTML for a <glosslist>.

Here is an example:

<glosslist>
    <glossentry>
    <glossterm>compute nodes</glossterm>
    <glossdef>
        <para>
        A node which receives tasks to be run as jobs.
        </para>
        <glossseealso linkend="gl-head-nodes"/>
    </glossdef>
    </glossentry>
    <glossentry xml:id="gl-head-nodes">
    <glossterm>head node(s)</glossterm>
    <glossdef>
        <para>
        A node which receives tasks to be run as jobs.
        </para>
    </glossdef>
    </glossentry>
</glosslist

After transforming it to HTML, you get this output:

glosslist-in-html

So to summarize the issues here:

  1. the spaces between a glossdef and the next glossterm is broken.
  2. ~the reference of the glossseealso does not work.~ (see "Update" below)

For PDF, ~only point 2 is broken.~ it works.

Expected Behaviour

I would propose to use the same rendering than in PDF: render it as a variablelist. We have that already in PDF, so we only need to adapt it to HTML.


Update: I made a mistake: it should be <glossseealso otherterm="..."/> instead of <glossseealso linkend="gl-head-nodes"/>. If I use this, point 2 from above works.

janajaeger commented 2 years ago

What's the status of this PR, @tomschr ? Can you close this one?

tomschr commented 1 year ago

Needs to be investigated further.