Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website - Mailing list pgadmin-hackers

Thanks, applied with a few changes for consistency in the titles and 404 handling.

On Wed, Aug 26, 2020 at 8:23 AM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:

Hi Dave/Team,

Updated the implementation for the set title of the page, for this, I have done a few changes in the flow.

1. Earlier we are using {% include %} for showing pages in the base page (styleguide/index.html) for a style guide but in the Django template we can't use {% block %} statement if we are adding page using "{% include %}".

2. To overcome this issue now using {% extends %} instead of  {% include %},  For this added dynamic URL generation in views so instead of loading "index.html" of style guide we are loading specific component HTML file (e.g: typography.html) and that template extends the "index.html" of the style guide.

PFA v6 patch for the same.

If required any changes please let me know.

Regards,

Nikhil Mohite.

On Tue, Aug 25, 2020 at 7:09 PM Dave Page <dpage@pgadmin.org> wrote:

Hi

On Tue, Aug 25, 2020 at 12:49 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:

PFA v5 patch for the same.

if required any updates in this please let me know.

Hmm, yeah - that doesn't work for me I'm afraid; it's mixing content with code.

The titles should be defined in the HTML pages themselves. We do that everywhere else by including a template block like the following in the lowest level content:

{% block title %}Contributing{% endblock %}

And then in base.html:

<title>{% block title %}{% endblock %}</title>

--

Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EDB: http://www.enterprisedb.com

--

Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EDB: http://www.enterprisedb.com