On 11 Jun 2018, at 19:52, Jonathan S. Katz <jkatz@postgresql.org> wrote:
On Jun 11, 2018, at 1:47 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Since we moved to the new website design, the mailing list archives
display all messages with leading indentation suppressed. This can
be pretty awful, eg in the code fragment here:
https://www.postgresql.org/message-id/20180611170812.7ypf6xs6dlteo57j@alap3.anarazel.de
Looking closer, I think it may not just be leading indentation that
gets lost; seems like *any* whitespace string gets smashed to a
single space, cf followup at
https://www.postgresql.org/message-id/7382.1528737252%40sss.pgh.pa.us
which certainly does not look like it did in my outbox.
I assume this is just an ill-considered stylesheet choice somewhere
... can we fix it please?
Hmm…agreed. I’m going to check the old code to see how that was handled. This
may have occurred when trying to make the archives readable across different
viewports.
The below diff will retain the whitespace included the post. It doesn’t seemto make it less readable on smaller viewports (although more scrolling), but Ionly tested quickly so more input is welcome.
I did some testing locally and it does solve the whitespace problem, but
introduces the following:
1. The line heights are not a bit larger, which could introduce the return of
the “scrolling” problem.
2. We lose the mobile-friendly wrapping that was introduced.
Looking at our mobile traffic numbers, I’d be inclined to solve the immediate
issue of the whitespace problem and make it a bit harder to use the archives on
mobile temporarily while looking for a better solution. In other words, we don’t
have that much mobile traffic in the archives.
I can tweak the line heights the make the scrolling not as bad, if not the same.
I’ll experiment and once it looks like before push / commit.
Jonathan
