Re: doc: add missing "id" attributes to extension packaging page
Brar Piening <brar@gmx.de>
From: Brar Piening <brar@gmx.de>
To: Peter Eisentraut <peter.eisentraut@enterprisedb.com>,
Alvaro Herrera <alvherre@alvh.no-ip.org>, "Karl O. Pinc" <kop@karlpinc.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, vignesh C <vignesh21@gmail.com>,
Ian Lawrence Barwick <barwick@gmail.com>,
pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2023-04-06T14:19:30Z
Lists: pgsql-hackers
Attachments
- 002-add-discoverable-id-style_v1.pgweb.patch (text/plain) patch v1
- 001-make_html_ids_discoverable_v5.postgresql.patch (text/plain) patch v5
On 06.04.2023 at 11:06, Peter Eisentraut wrote: > On 04.04.23 21:52, Brar Piening wrote: >>> The XSLT implementation looks sound to me. It would be a touch better >>> if it had some comments about which parts of the templates were copied >>> from upstream stylesheets and which were changed. There are examples >>> of such commenting in the existing customization layer. Also, avoid >>> introducing whitespace differences during said copying. >> >> I will amend the patch if we agree that this is the way forward. > > Ok, it appears everyone agrees that this is the correct approach. > Please post an updated patch. There have been so many partial patches > posted recently, I'm not sure which one is the most current one and > who is really the author. > Attached are two patches: 001-make_html_ids_discoverable_v5.postgresql.patch which needs to be applied to the postgresql repository. It adds the XSLT to generate the id links and the CSS to hide/display them. I've added comments as suggested above. 002-add-discoverable-id-style_v1.pgweb.patch which needs to be applied to the pgweb repository. It adds the CSS to the offical documentation site. At the moment (commit 983ec23007) there are no missing ids, so the build should just work after applying the patch but, as we already know, this may change with every commit that gets added. Reviewer is Karl O. Pink Author is Brar Piening (with some additions from Karl O. Pink) Best regards, Brar
Commits
-
doc: Make HTML ids discoverable
- e2922702a302 16.0 landed
-
Add missing XML ID attribute
- a34901dd03ed 16.0 landed
-
postgres_fdw: Add support for parallel abort.
- 983ec23007bd 16.0 cited
-
Add missing XML ID attributes
- 3077324b03e8 16.0 landed
-
Add XML ID attributes to create_subscription.sgml.
- de5a47af2d80 16.0 cited
-
Stop recommending auto-download of DTD files, and indeed disable it.
- 969509c3f2e3 16.0 cited
-
Doc: add XML ID attributes to <sectN> and <varlistentry> tags.
- 78ee60ed84bb 16.0 landed
-
Allow left join removals and unique joins on partitioned tables
- 3c569049b7b5 16.0 cited
-
Improve GIN cost estimation
- cd9479af2af2 16.0 cited