Do not escape urls where it is not needed

Signed-off-by: Alexander Borsuk <[email protected]>
Grreby · Dec 24, 2022 · 80153d7 · 80153d7
1 parent 5cf6282
commit 80153d7
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/templates/base.html b/templates/base.html
@@ -30,7 +30,7 @@
   {%- endif %}
   {%- endif %}
   <meta property="og:image" content="{{ get_url(path=preview_image) }}">
-  <meta property="og:url" content="{{ current_url }}">
+  <meta property="og:url" content="{{ current_url | safe }}">
   <meta property="og:type" content="website">
   <meta property="og:title" content="{{ resource.title }}">
   {%- if resource.description -%}

diff --git a/templates/bottom_menu.html b/templates/bottom_menu.html
@@ -13,7 +13,7 @@
   {% endif %}
 
   {% if page.extra.menu_title %}
-  {% if resource.path != page.path %}<a href="{{ page.permalink | replace(from=config.base_url, to="") }}">{% endif %}
+  {% if resource.path != page.path %}<a href="{{ page.permalink | replace(from=config.base_url, to="") | safe }}">{% endif %}
     <span>{{ page.extra.menu_title }}</span>
     {%- if resource.path != page.path %}</a>{% endif %}
   &nbsp;

diff --git a/templates/language_selector.html b/templates/language_selector.html
@@ -17,7 +17,7 @@
     {% for translation in resource.translations | sort(attribute="lang") %}
     {% if lang != translation.lang %}
     <li class="lang-menu-item" role="menuitem">
-      <a class="lang-menu-link" onclick="return onLanguageClick('{{ translation.lang }}');" href="{{ translation.permalink | replace(from=config.base_url, to="") }}">
+      <a class="lang-menu-link" onclick="return onLanguageClick('{{ translation.lang }}');" href="{{ translation.permalink | replace(from=config.base_url, to="") | safe }}">
         {{ trans(key="language", lang=translation.lang) }}
       </a>
     </li>

diff --git a/templates/top_menu.html b/templates/top_menu.html
@@ -4,7 +4,7 @@
   {% set langPathPrefix = '/' ~ lang %}
 {% endif %}
 <nav id="menu" role="navigation">
-  <a href="{{ langPathPrefix }}/"><span class="top_menu_logo"></span></a>
+  <a href="{{ langPathPrefix | safe }}/"><span class="top_menu_logo"></span></a>
   <span>
     {% for page in [
       get_section(path="news/_index.md"),
@@ -20,7 +20,7 @@
       {% endif %}
     {% endif %}
     <span class="top_menu_{{ page.path | split(pat="/") | reverse | nth(n=1) }} top_menu_item{% if page.path != resource.path %}">
-      <a href="{{ page.permalink | replace(from=config.base_url, to="") }}">{% else %}_active">{% endif %}
+      <a href="{{ page.permalink | replace(from=config.base_url, to="") | safe }}">{% else %}_active">{% endif %}
         <span>{{ page.extra.menu_title }}</span>
         {%- if page.path != resource.path %}</a>{% endif %}
     </span>