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/
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 |