Editor mode in Mallard?

On Wed, Mar 7, 2012 at 19:54, Shaun McCance <shaunm at gnome.org> wrote:
> On Wed, 2012-03-07 at 17:29 +0100, Philip Chimento wrote:
>> On Wed, Mar 7, 2012 at 16:01, Shaun McCance <shaunm at gnome.org> wrote:
>> > On Sun, 2012-02-26 at 20:42 +0100, Philip Chimento wrote:
>> >> I'd like to show some indication on pages that have
>> >> <revision status="incomplete"/> or whatever in their <info> element.
>> >> According to the docs, there's a parameter mal2html.editor_mode that I
>> >> should set. I have the line in my custom XSLT:
>> >>
>> >> <param name="mal2html.editor_mode" select="true()"/>
>> >>
>> >> But this doesn't seem to do anything. Neither does editing
>> >> mal2html-page.xsl directly and changing false() to true(). But if I
>> >> look in the code, there should be something output in
>> >> mal2html.editor.banner, which is called from html.body.mode.
>> >
>> > Do you have any revision elements with a status attribute in your
>> > info element?
>>
>> I have things like
>>
>> <revision status="incomplete"/>
>>
>> in my <info> elements on several subpages. But not on my index.page.
>> Does that mean the revision status is set for the entire help
>> document, and not per-page?
>
> No, revisions are always set per-page. There's no such thing as a
> per-document revision in Mallard, although I tend to use revision
> on the index page as if it were the overall document status.
>
> It looks like the code in yelp-xsl only selects revision elements
> that have a date attribute. That seemed wrong to me, so I've fixed
> it in yelp-xsl git master just now. If you don't feel like grabbing
> the XSLT from git, just use date attributes.

That did it, excellent!

(By the way, if anyone on the list is interested in my Mallard project
in progress, go to
http://ptomato.github.com/advanced-gtk-techniques/html/index.html or
https://github.com/ptomato/advanced-gtk-techniques for the code)

Best,
-- 
Philip