#5 Active page does not work, but even if it did the template has no style class for it

Open
opened 6 years ago by vaeringjar · 4 comments

Basically two problems, though minor:

  • pagePrime does not actually detect the current page.
  • The style sheet has no class named "active"
$ grep -in "pagePrime\|active" themes/pelican/templates/*
themes/pelican/templates/base.html:52:            <li{% if page == pagePrime %} class="active"{% endif %}><a href="{{ link }}">{{ title }}</a></li>
themes/pelican/templates/base.html:64:            <li{% if page == pagePrime %} class="active"{% endif %}><a href="{{ SITEURL }}/{{ page.url }}">{{ page.title }}</a></li>
themes/pelican/templates/base.html:70:            <li{% if cat == category %} class="active"{% endif %}><a href="{{ SITEURL }}/{{ cat.url }}">{{ cat }}</a></li>

Basically two problems, though minor: * pagePrime does not actually detect the current page. * The style sheet has no class named "active" ``` $ grep -in "pagePrime\|active" themes/pelican/templates/* themes/pelican/templates/base.html:52: <li{% if page == pagePrime %} class="active"{% endif %}><a href="{{ link }}">{{ title }}</a></li> themes/pelican/templates/base.html:64: <li{% if page == pagePrime %} class="active"{% endif %}><a href="{{ SITEURL }}/{{ page.url }}">{{ page.title }}</a></li> themes/pelican/templates/base.html:70: <li{% if cat == category %} class="active"{% endif %}><a href="{{ SITEURL }}/{{ cat.url }}">{{ cat }}</a></li> ```
zPlus commented 6 years ago
Owner

What are the "expected behavior" and "actual/current behavior"?

What are the "expected behavior" and "actual/current behavior"?
vaeringjar commented 6 years ago
Owner

The original template, which was a fork itself, had a way alter the menu to highlight or underline the text/link for the current page.

The original template, which was a fork itself, had a way alter the menu to highlight or underline the text/link for the current page.
zPlus commented 6 years ago
Owner

There are only two pages, "TODO" and "Why ?", correct?

There are only two pages, "TODO" and "Why ?", correct?
vaeringjar commented 6 years ago
Owner

This might not make any sense anymore for this template. The original template before the fork didn't really have it either. I'll think about it for a few more days, but I think just removing the references makes more sense.

This might not make any sense anymore for this template. The original template before the fork didn't really have it either. I'll think about it for a few more days, but I think just removing the references makes more sense.
Sign in to join this conversation.
No Label
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.