Fixed Template 'navigation' misses terms conditional

AlexT

Well-known member
The header/quick links navigation of the help section misses the conditional to display a link to the Terms and Rules.

HTML:
<xen:hook name="navigation_tabs_help">
    <li><a href="{xen:link help/smilies}">{xen:phrase smilies}</a></li>
    <li><a href="{xen:link help/bb-codes}">{xen:phrase bb_codes}</a></li>
    <li><a href="{xen:link help/trophies}">{xen:phrase trophies}</a></li>
    <li><a href="{xen:link help/cookies}">{xen:phrase cookie_usage}</a></li>
</xen:hook>
should be

HTML:
<xen:hook name="navigation_tabs_help">
    <li><a href="{xen:link help/smilies}">{xen:phrase smilies}</a></li>
    <li><a href="{xen:link help/bb-codes}">{xen:phrase bb_codes}</a></li>
    <li><a href="{xen:link help/trophies}">{xen:phrase trophies}</a></li>
    <li><a href="{xen:link help/cookies}">{xen:phrase cookie_usage}</a></li>
<xen:if is="{$tosUrl}">
    <li><a href="{$tosUrl}">{xen:phrase terms_and_rules}</a></li>
</xen:if>   
</xen:hook>
 

Brogan

XenForo moderator
Staff member
That hasn't been present since the first beta, I always assumed it was intentional.
 

Jeremy

Well-known member
This is most likely intentional because I don't believe $tosUrl is set in the navigation template. I'm at work so I can't look stuff up, but you could possibly reference the tosUrl option directly to display it in the main navigation.
 

AlexT

Well-known member
I thought whether it was intentionally, but IMHO it doesn't seem logical. I would expect the list of links in the navigation to resemble the list of links on the left side of the help section.

OK, you could argue that there is already a link to the Tos in the lower right footer, but that doesn't mean that there shouldn't be a link in the navigation.
 

Brogan

XenForo moderator
Staff member
I did actually edit mine to include it from day 1, as it seemed a bit odd having it in the page but not on the nav bar.
 

Jeremy

Well-known member
I lied. It is exposed. I'm in agreement with AlexT here. Might also add in the <!-- slot: --> comments to match the main help page template_hook.

I went through and tested it. Needed a break from work work.
 
Top