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

From Nikhil Mohite
Subject Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website
Date
Msg-id CAOBg0APsOizjnj-D2dCAfwbbobwgJ4mFcjtyJEuwh3QborQiUQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
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

Attachment

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin][Patch] - Housekeeping #5332 - [Code Coverage] Improve API test cases for Tables, Columns and Constraints (Index, Foreign Key, Check, Exclusion)
Next
From: Dave Page
Date:
Subject: Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website