Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add active class to cms content pages in mainnavbar #126

Closed
wants to merge 2 commits into from

Conversation

Projects
None yet
4 participants
@bpmusicDE
Copy link
Contributor

commented Jun 26, 2018

No description provided.

@CLAassistant

This comment has been minimized.

Copy link

commented Jun 26, 2018

CLA assistant check
All committers have signed the CLA.

@@ -31,7 +31,7 @@
[{foreach from=$oxcmp_categories item="ocat" key="catkey" name="root"}]
[{if $ocat->getIsVisible()}]
[{foreach from=$ocat->getContentCats() item="oTopCont" name="MoreTopCms"}]
<li>
<li [{if $oContent->oxcontents__oxloadid->value===$oTopCont->oxcontents__oxloadid->value}]class="active"[{/if}]>

This comment has been minimized.

Copy link
@Crease29

Crease29 Jun 26, 2018

Contributor

I would change the formatting to: <li[{if $oContent->oxcontents__oxloadid->value === $oTopCont->oxcontents__oxloadid->value}] class="active"[{/if}]>

This comment has been minimized.

Copy link
@bpmusicDE

bpmusicDE Jun 26, 2018

Author Contributor

ok

@Sieg Sieg added the in progress label Jul 19, 2018

@Sieg Sieg self-assigned this Jul 19, 2018

@Sieg Sieg added the Ready to merge label Jul 19, 2018

@Sieg

This comment has been minimized.

Copy link
Contributor

commented Aug 17, 2018

Thank you @bpmusicDE for your pull request! i have merged it recently at 8dce4b4

@Sieg Sieg closed this Aug 17, 2018

@Sieg Sieg removed the in progress label Aug 17, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.