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 <dp...@pgadmin.org> wrote:
> Hi
>
> On Tue, Aug 25, 2020 at 12:49 PM Nikhil Mohite <
> nikhil.moh...@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
>
>
RM_5690_v6.patch
Description: Binary data
