Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

PLAT-459 nav target #491

Merged
merged 4 commits into from
Sep 7, 2016
Merged

PLAT-459 nav target #491

merged 4 commits into from
Sep 7, 2016

Conversation

gusliedke
Copy link
Contributor

@@ -44,7 +44,9 @@
{% if menu_level == 1 %}
{# Duplicate menu link for UX, to be populated by JS #}
<li class="menu-item menu-item--duplicate">
{{ link(item.title, item.url) }}
<a href="/">
<span class="menu-item__text">{{menu_level}}</span>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gusliedke Need to remove this, the JS is already adding the span. Or remove it from the JS, either way.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed from template

@gusliedke gusliedke added this to the 1.10.1 milestone Sep 7, 2016
@gusliedke gusliedke added the in QA label Sep 7, 2016
@gusliedke gusliedke modified the milestones: 1.11, 1.10.1 Sep 7, 2016
@AndyEPhipps AndyEPhipps merged commit 65e8f70 into develop Sep 7, 2016
@AndyEPhipps AndyEPhipps deleted the feature/PLAT-459_nav_target branch September 7, 2016 12:02
@Saphyel Saphyel changed the title Feature/plat 459 nav target PLAT-459 nav target Sep 7, 2016
This was referenced Sep 7, 2016
@pvhee pvhee mentioned this pull request Jan 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants