From 68b4fe1726a56019febced2dff9da95d71ea51d7 Mon Sep 17 00:00:00 2001
From: Jorran de Wit <jorrandewit@outlook.com>
Date: Tue, 16 May 2017 10:50:20 +0200
Subject: [PATCH] Minor improvement frontpage

---
 .../static/scipost/assets/css/_general.scss   |  5 ++++
 scipost/static/scipost/assets/css/_nav.scss   | 17 +++++++++++++
 scipost/static/scipost/assets/css/_type.scss  |  6 +++++
 scipost/templates/scipost/index.html          | 24 +++++++++----------
 4 files changed, 39 insertions(+), 13 deletions(-)

diff --git a/scipost/static/scipost/assets/css/_general.scss b/scipost/static/scipost/assets/css/_general.scss
index e1758c796..a27786c26 100644
--- a/scipost/static/scipost/assets/css/_general.scss
+++ b/scipost/static/scipost/assets/css/_general.scss
@@ -23,3 +23,8 @@ body #MathJax_Message {
     color: #002b49;
     opacity: 0.9;
 }
+
+.inner-container-smaller {
+    max-width: 1100px;
+    margin: 0 auto;
+}
diff --git a/scipost/static/scipost/assets/css/_nav.scss b/scipost/static/scipost/assets/css/_nav.scss
index d24c006ea..4519db303 100644
--- a/scipost/static/scipost/assets/css/_nav.scss
+++ b/scipost/static/scipost/assets/css/_nav.scss
@@ -48,3 +48,20 @@
         }
     }
 }
+
+nav.submenu {
+    margin-top: -0.75rem;
+    margin-bottom: 1rem;
+    padding: 0.75rem 1rem;
+    background-color: #f9f9f9;
+    border-radius: 0;
+
+    .item {
+        padding: 0 0.5rem;
+        border-right: 2px solid #fff;
+    }
+    .item:first-child,
+    .item:last-child {
+        border-right: 0;
+    }
+}
diff --git a/scipost/static/scipost/assets/css/_type.scss b/scipost/static/scipost/assets/css/_type.scss
index 327ab7b99..408d1fc73 100644
--- a/scipost/static/scipost/assets/css/_type.scss
+++ b/scipost/static/scipost/assets/css/_type.scss
@@ -27,6 +27,12 @@ h1 > a {
     color: $scipost-darkblue;
 }
 
+.orange-underline {
+    line-height: 1.3;
+    border-bottom: 2px solid $scipost-orange;
+    padding-bottom: 0.75rem;
+}
+
 .highlight {
     background-color: #f4f4f4;
     border-radius: 1.4px;
diff --git a/scipost/templates/scipost/index.html b/scipost/templates/scipost/index.html
index f0d2d32db..66fde08fe 100644
--- a/scipost/templates/scipost/index.html
+++ b/scipost/templates/scipost/index.html
@@ -4,25 +4,23 @@
 
 {% block breadcrumb %}
     {% if journals|length > 1 %}
-    <nav class="breadcrumb py-md-2 px-0 as-submenu">
-        <div class="container">
-            <span class="breadcrumb-item">Go direct to Journal:</span>
-            {% for journal in journals %}
-                <a href="{{journal.get_absolute_url}}" class="breadcrumb-item {% if forloop.counter == 1 %} ml-2 no-divider{% endif %}">{{journal}}</a>
-            {% endfor %}
-        </div>
+    <nav class="submenu">
+        <span class="item">Go directly to Journal:</span>
+        {% for journal in journals %}
+            <a href="{{journal.get_absolute_url}}" class="item">{{journal}}</a>
+        {% endfor %}
     </nav>
     {% endif %}
 {% endblock %}
 
 {% block content %}
-
+<div class="inner-container-smaller">
     <div class="row pt-4">
         <div class="col-md-7">
             <!-- Latest publications -->
             <div class="card card-grey">
                 <div class="card-header">
-                    <h1 class="card-title mb-0"><a href="{% url 'journals:journals' %}">Latest Publications</a></h1>
+                    <h2 class="card-title mb-0"><a href="{% url 'journals:journals' %}" class="text-black">Latest Publications</a></h2>
                 </div>
                 <div class="card-block pt-0">
                   <ul class="list-group list-group-flush">
@@ -41,7 +39,7 @@
             <!-- Latest submissions -->
             <div class="card card-grey">
                 <div class="card-header">
-                    <h1 class="card-title mb-0"><a href="{% url 'submissions:submissions' %}">Latest Submissions</a></h1>
+                    <h2 class="card-title mb-0"><a href="{% url 'submissions:submissions' %}" class="text-black">Latest Submissions</a></h2>
                 </div>
                 <div class="card-block pt-0">
                   <ul class="list-group list-group-flush">
@@ -56,11 +54,11 @@
         </div>
 
         <div class="col-md-5">
-            <div class="pl-lg-5 pr-lg-1">
+            <div class="pl-lg-5">
                 <!-- Info -->
                 <div class="card">
                     <div class="card-block text-center pt-0">
-                        <h3 class="m-0" style="line-height: 1.3;">SciPost is a complete scientific publication portal managed by active professional scientists.</h3>
+                        <h3 class="m-0 orange-underline" style="line-height: 1.3;">SciPost is a complete scientific publication portal managed by active professional scientists.</h3>
                     </div>
                 </div><!-- End info -->
 
@@ -70,7 +68,7 @@
                         <h2 class="card-title mb-0"><a href="/issues/" class="text-black">Issues</a></h2>
                     </div>
                     <div class="card-block">
-                        <ul class="m-0">
+                        <ul class="m-0 pl-4">
                             {% for issue in issues %}
                                 <li>
                                     <a href="{{issue.get_absolute_url}}">{{issue}}</a>
-- 
GitLab