Re: "Fast Forward" links don't seem to be correct

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: pgsql-docs(at)postgresql(dot)org
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: "Fast Forward" links don't seem to be correct
Date: 2006-10-22 14:54:49
Message-ID: 200610221654.50518.peter_e@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs

Tom Lane wrote:
> If you go to a regular, non-chapter-heading page in the PG docs, such
> as
> http://developer.postgresql.org/pgdocs/postgres/extend-type-system.ht
>ml and try the "Fast Backward" link, it goes to that chapter's heading
> page, which seems reasonable. But the "Fast Forward" link points to
> the same place. Shouldn't it lead to the *next* chapter, instead?

It probably should. The stylesheet code, however, doesn't seem to make
any attempts about it:

(define (next-major-component-chunk-element #!optional (elem (current-node)) (in-chain #f))
;; Return the next major component of the document that is not a descendant
;; of the starting element. This is essentially 'next-sibling' but skips
;; over things that aren't chunks.
(if (or (navigate-to? elem) in-chain)
(if (member (gi elem) (major-component-element-list)) ; <--- It's a chapter or other major chunk.
(if (node-list-empty? (node-list-first-element (follow elem)))
(next-major-component-chunk-element (parent elem))
(let ((nd (node-list-first-element (follow elem))))
(if (navigate-to? nd)
nd
(next-major-component-chunk-element nd #t))))
(ancestor-member elem (major-component-element-list))) ; <--- Else picks ancestor (should pick something after ancestor).
(empty-node-list)))

I tried to change that but my first attempts seem to make the build take
forever. I'll play around with it some more.

At least I think that's where it's happening ...

--
Peter Eisentraut
http://developer.postgresql.org/~petere/

In response to

Browse pgsql-docs by date

  From Date Subject
Next Message David Blewett 2006-10-23 17:55:05 8.1.5 Release Notes
Previous Message Tom Lane 2006-10-22 02:45:41 Re: Empty index in A4 version of 8.1 PDF documentation