Re: [PATCH] Change text direction of documentation pages

On 11/29/21 4:16 AM, Daniel Gustafsson wrote:

>> Overall OK with the approach, but would like to see how it renders.
>
> I don't have a local pgweb setup for now, so feel free to pick it up and play
> with it if you have time.

Fast forward to the future, I went and played around with the suggested
patch, i.e.:

- <title>PostgreSQL: Documentation: {{page.display_version}}:
{{page.title}}</title>
+ <title>{{page.title}} &mdash; PostgreSQL {{page.display_version}}
Documentation</title>

It looks OK...but I question having the chapter/section prefix in the
title, i.e.:

"7.2 Table Expressions -- PostgreSQL 10 Documentation"

(yes, I need to update my local copy of the docs).

I think:

"Table Expressions -- PostgreSQL 10 Documentation"

would be better, esp. from the SEO perspective. This would also mean
adjusting our Open Graph tags to account for it from a display
perspective as well. And writing a function to strip out the prefix.

However, this opens up a few things:

1. On the main doc page, it now reads something like "PostgreSQL 13.5
Documentation - PostgreSQL 13 Documentation." That should be simple
enough to adjust though.

2. On this page:

https://www.postgresql.org/docs/10/typeconv-overview.html

the title would then read "Overview -- PostgreSQL 10 Documentation",
which also seems off. So perhaps the general algorithm becomes:

"Page Title -- Chapter Name -- PostgreSQL NN Documentation"

which would make that:

"Overview -- Type Conversation -- PostgreSQL 10 Documentation"

So, I think this is a little more work. I would propose this:

- In the doc loader script, extract the "chapter" name out of the
provided information and store it in DocPage OR dynamically extract it
while rendering a documentation page. I'm thinking the latter for this.

- Have a "page title" in the documentation available without the
chapter/section prefix

- Set the page title to be something like "Title w/o Prefix &mdash;
Chapter &mdash; PostgreSQL NN Documentation", with title/chapter dropped
if they're not present.

Thoughts?

Thanks,

Jonathan