From: Tom Lane <tgl@sss.pgh.pa.us>
To: Daniel Gustafsson <daniel@yesql.se>
cc: pgsql-docs@lists.postgresql.org
Subject: Re: Change pg_attribute textual link to an actual link
In-reply-to: <0E8748E3-8B7D-445E-9ABA-09DA5C7345CC@yesql.se>
References: <0E8748E3-8B7D-445E-9ABA-09DA5C7345CC@yesql.se>
Comments: In-reply-to Daniel Gustafsson <daniel@yesql.se>
	message dated "Sun, 21 Oct 2018 22:29:58 +0200"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <80061.1540164441.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Sun, 21 Oct 2018 19:27:21 -0400
Message-ID: <80062.1540164441@sss.pgh.pa.us>
Precedence: bulk

Daniel Gustafsson <daniel@yesql.se> writes:
> In the System catalog/pg_attrdef documentation, there is a reference to
> pg_attribute with the text "see below".  That makes sense in the PDF ver=
sion
> (and similar renderings), but in the web version it seems a bit odd as
> pg_attribute is on the next page, not below.

> The attached patch changes it to an actual link instead, but simply remo=
ving
> the "see below" bit is another approach which isn't out of line with oth=
er
> parts of the documentation (I opted for link since I personally prefer t=
hat).

I agree with your change, but I wonder if the sentence after this one
doesn't need some work too.  It seems to predate ALTER COLUMN SET DEFAULT.
We could add a mention of that, or just stop trying to describe the
preconditions, and say something like "only columns for which a default
value has been explicitly set will have an entry here".

			regards, tom lane