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

Replaced emoji for special page nav link with text #12728

Merged
merged 9 commits into from
Aug 28, 2024
2 changes: 1 addition & 1 deletion network-api/networkapi/templates/tags/horizontal_nav.html
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
{% for entry in menu_pages %}
{% pageurl entry.page as entry_page_url %}
{% if entry.depth < 2 %}
<a class="multipage-link {% if current_page_url == entry_page_url %}active{% endif %} {% if entry.restriction != None %}multipage-{{ entry.restriction }}-restricted{% endif %} {% if not entry.page.live %}multipage-draft{% endif %}" href="{% pageurl entry.page %}">{{ entry.menu_title }}</a>
<a class="multipage-link {% if current_page_url == entry_page_url %}active{% endif %}" href="{% pageurl entry.page %}">{{ entry.menu_title }} {% include "tags/multipage_menu_link_note.html" with menu_entry=entry %}</a>
{% endif %}
{% endfor %}
</div>
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
{% load i18n %}

{% with class="tw-text-red-80 tw-text-sm tw-ml-1" note_for_translators="This note is internal facing only and is only visible to our CMS logged in users." %}
{% comment %}These notes are internal facing only. Only logged in users can see them.{% endcomment %}

{% if menu_entry.restriction == "login" %}
<span class="{{ class }}">{% trans "(logged in users only)" context note_for_translators %}</span>
{% elif menu_entry.restriction == "groups" %}
<span class="{{ class }}">{% trans "(restricted to specific groups)" context note_for_translators %}</span>
{% elif menu_entry.restriction == "password" %}
<span class="{{ class }}">{% trans "(password protected)" context note_for_translators %}</span>
{% endif %}

{% if not menu_entry.page.live %}
<span class="{{ class }}">{% trans "(draft)" context note_for_translators %}</span>
{% endif %}
mmmavis marked this conversation as resolved.
Show resolved Hide resolved

{% endwith %}
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
{% for entry in menu_pages %}
{% pageurl entry.page as entry_page_url %}
<div class="multipage-link-wrapper">
<a class="multipage-link {% if current_page_url == entry_page_url %}active{% endif %} {% if entry.restriction != None %}multipage-{{ entry.restriction }}-restricted{% endif %} {% if not entry.page.live %}multipage-draft{% endif %}" href="{% pageurl entry.page %}">{% if entry.depth == 2 %}→ {% endif %}{{ entry.menu_title }}
<a class="multipage-link {% if current_page_url == entry_page_url %}active{% endif %} {% if entry.restriction != None %}multipage-{{ entry.restriction }}-restricted{% endif %} {% if not entry.page.live %}multipage-draft{% endif %}" href="{% pageurl entry.page %}">{% if entry.depth == 2 %}→ {% endif %}{{ entry.menu_title }} {% include "tags/multipage_menu_link_note.html" with menu_entry=entry %}
</a>
</div>
{% endfor %}
Expand Down
16 changes: 0 additions & 16 deletions source/sass/components/multipage-nav.scss
Original file line number Diff line number Diff line change
Expand Up @@ -36,20 +36,4 @@
@apply tw-mb-12;
}
}

.multipage-login-restricted::after {
content: "🙈";
}

.multipage-groups-restricted::after {
content: "👥";
}

.multipage-password-restricted::after {
content: "🔒";
}

.multipage-draft::after {
content: "🐣";
}
}
Loading