From 7cf170e000b3233c626a0de3af0d3609b94222a9 Mon Sep 17 00:00:00 2001
From: Pablo Vazquez <pablo@blender.org>
Date: Thu, 15 Aug 2024 15:32:35 +0200
Subject: [PATCH] Films: Navigation bar adjustments

* Remove link to "Home". It can be confused with the film home, and it
  is already available when clicking on the giant "Blender Studio" link
  in the top bar.

* Make film title bold.
* Add separator between film title and other pages.
---
 films/templates/films/base_films.html | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/films/templates/films/base_films.html b/films/templates/films/base_films.html
index fe3f4823..ba1b2e60 100644
--- a/films/templates/films/base_films.html
+++ b/films/templates/films/base_films.html
@@ -14,20 +14,13 @@
   <header class="navbar navbar-secondary" role="navigation">
     <div class="container">
       <ul class="navbar-nav">
-        <li class="nav-item nav-parent show-on-scroll" data-link="/">
-          <a class="nav-link" href="/">Home</a>
-        </li>
         <li class="nav-item" data-link="{% url 'film-detail' film_slug=film.slug %}">
-          <a class="nav-link" href="{% url 'film-detail' film_slug=film.slug %}">{{ film.title }}</a>
+          <a class="nav-link" href="{% url 'film-detail' film_slug=film.slug %}"><strong>{{ film.title }}</strong></a>
         </li>
+        <li class="nav-separator"><i class="i-chevron-right"></i></li>
         <li class="nav-item {% if current_collection %}active{% endif %}" data-link="{% url 'film-gallery' film_slug=film.slug %}">
           <a class="nav-link" href="{% url 'film-gallery' film_slug=film.slug %}">Content Gallery</a>
         </li>
-        {% for flatpage in film.flatpages.all %}
-          <li class="nav-item" data-link="{% url 'film-flatpage' film_slug=film.slug page_slug=flatpage.slug %}">
-            <a class="nav-link" href="{% url 'film-flatpage' film_slug=film.slug page_slug=flatpage.slug %}">{{ flatpage.title|title }}</a>
-          </li>
-        {% endfor %}
         {% if film.show_production_logs_nav_link %}
           <li class="nav-item {% if date_list or production_log.name %}active{% endif %}">
             <a class="nav-link" href="{% url 'film-production-logs' film_slug=film.slug %}">Production Logs</a>
@@ -38,6 +31,11 @@
           </li>
           {% endif %}
         {% endif %}
+        {% for flatpage in film.flatpages.all %}
+          <li class="nav-item" data-link="{% url 'film-flatpage' film_slug=film.slug page_slug=flatpage.slug %}">
+            <a class="nav-link" href="{% url 'film-flatpage' film_slug=film.slug page_slug=flatpage.slug %}">{{ flatpage.title|title }}</a>
+          </li>
+        {% endfor %}
         {% if user_has_production_credit or credit %}
           <li class="nav-item" data-link="{% url 'production-credit' film_slug=film.slug %}">
             <a href="{% url 'production-credit' film_slug=film.slug %}" class="nav-link">
-- 
2.30.2