Feedback-ID: ia2694551:Fastmail
Date: Thu, 9 Mar 2023 10:27:28 +0100
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: "Karl O. Pinc" <kop@karlpinc.com>
Cc: w^3 <pgsql-www@lists.postgresql.org>
Subject: Re: Doc: Rework contrib appendix -- informative titles, tweaked
 sentences
Message-ID: <20230309092728.3samipw726tf3fa6@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="jffuhrgohs4dd7f7"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20230122144246.0ff87372@slate.karlpinc.com>
Archived-At: 
 <https://www.postgresql.org/message-id/20230309092728.3samipw726tf3fa6%40alvherre.pgsql>
Precedence: bulk


--jffuhrgohs4dd7f7
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit

Hello pgsql-web,

We're looking to improve the contrib docs with a list of trusted
extensions.  In order for that look more presentable, Karl has come up
with the idea of using a <simplelist> table with multiple columns.  That
would normally look quite terrible because the cell contents are too
close to one another, so he came up with the idea of increasing the
padding, as shown in this patch.

I think this is good thing, as it can help us use tabular <simplelist>
in other places too.

This change requires to change main Postgres doc/src/sgml/stylesheet.css 
as Karl suggested here:

On 2023-Jan-22, Karl O. Pinc wrote:

> Actually, this CSS, added to doc/src/sgml/stylesheet.css,
> makes the column spacing look pretty good:
> 
> /* Adequate spacing between columns in a simplelist non-inline table */
> table.simplelist td { padding-left: 2em; padding-right: 2em; }
> 
> (No point in specifying table, since td only shows up in tables.)
> 
> Note that the default simplelist type value is "vert", causing a 1
> column vertical display.  There are a number of these in the
> documenation. I kind of like what the above css does to these
> layouts.  An example would be the layout in
> doc/src/sgml/html/datatype-boolean.html, which is the "Data Types"
> section "Boolean Type" sub-section.

... but in addition it needs the pgweb CSS to be updated to match, as in
the attached patch.

What do you think?

-- 
Álvaro Herrera               48°01'N 7°57'E  —  https://www.EnterpriseDB.com/
"The eagle never lost so much time, as
when he submitted to learn of the crow." (William Blake)

--jffuhrgohs4dd7f7
Content-Type: text/x-diff; charset=us-ascii
Content-Disposition: attachment;
	filename="0001-Add-padding-to-table.simplelist-for-more-readable-ou.patch"

From bb4a5388cbc373789c7548a4a390e37bbab8dbec Mon Sep 17 00:00:00 2001
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
Date: Thu, 9 Mar 2023 10:22:23 +0100
Subject: [PATCH] Add padding to table.simplelist for more readable output

Suggested by Karl Pinc
---
 media/css/main.css | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/media/css/main.css b/media/css/main.css
index 723366c9..fd082ae4 100644
--- a/media/css/main.css
+++ b/media/css/main.css
@@ -1169,6 +1169,12 @@ code,
   color: var(--doccontent-code-fg-color) !important;
 }
 
+/** Simplelist */
+#docContent table.simplelist td {
+	padding-left: 2em;
+	padding-right: 2em;
+}
+
 /**
   * Various callout boxes for docs, including warning, caution, note, tip
   */
-- 
2.30.2


--jffuhrgohs4dd7f7--