Merge pull request #134 from noesya/fix-publications

cleanup publication

assets/sass/_theme/sections/persons.sass

@@ -204,17 +204,6 @@ ol.persons--list
         margin-top: $spacing4
         ul
             @include list-reset
-        .publication
-            border-bottom: 1px solid $color-border
-            padding-bottom: $spacing1
-            padding-top: $spacing0
-            a
-                margin-bottom: calc(#{$spacing0}/2)
-                text-decoration: none
-            .ref
-                color: $color-text-alt
-                margin-top: 0
-                @include small
     @include media-breakpoint-down(lg)
         .roles
             margin-top: $spacing2

assets/sass/_theme/sections/publications.sass

@@ -12,41 +12,56 @@
         b, span
             display: block
 
+.publication
+    border-bottom: 1px solid $color-border
+    padding-bottom: $spacing0
+    padding-top: $spacing0
+    a
+        display: block
+        text-decoration: none
+
+    .publication-ref
+        @include small
+        color: $color-text-alt
+        margin-top: half($spacing0)
+    .publication-meta
+        @include meta
+
 .publications
     margin-top: $spacing3
-    article
+
+.publications__section
+    .publication
         padding-bottom: $spacing2
         padding-top: $spacing2
-        border-bottom: 1px solid $color-border
         position: relative
+        p
+            padding-right: $spacing2
+        @include media-breakpoint-up(desktop)
+            @include grid(3)
+            align-items: baseline
         @include media-breakpoint-down(desktop)
             padding-right: $spacing2
-            a + .meta
+            a + p
                 margin-top: $spacing1
             a::before
                 margin-top: px2rem(-6)
-        @include media-breakpoint-up(desktop)
-            @include grid(3)
-            align-items: baseline
+        .publication-ref
+            display: none
         a
             @include h3
             grid-column: 1 / 3
-            display: block
-            text-decoration: none
             @include stretched-link(after)
             &:hover
                 color: $color-accent
             @include icon-block(arrow-right, before)
                 position: absolute
                 right: 0px
-        .meta
-            padding-right: $spacing2
 
 .publications__page
     .hero
         h1, hgroup
             width: 100%
-
     .document-content > .container
         @include media-breakpoint-up(desktop)
             @include grid

assets/sass/_theme/sections/researchers.sass

+
+.researchers__term, .persons__page
+    .publication
+        .publication-meta
+            display: none

layouts/partials/persons/publications.html

@@ -5,13 +5,7 @@
   <ul>
     {{ range first 3 .publications }}
       <li>
-        <article class="publication">
-          {{ $title := .Title }}
-          <a href="{{ .Permalink }}" title="{{ safeHTML (i18n "commons.more_aria" (dict "Title" $title)) }}">{{ partial "PrepareHTML" $title }}</a>
-          <p class="meta">
-            {{ partial "PrepareHTML" .Params.ref }}
-          </p>
-        </article>
+        {{ partial "publications/publication.html" . }}
       </li>
     {{ end }}
   </ul>

layouts/partials/publications/publication.html

 <article class="publication">
   {{ $title := .Title }}
   <a href="{{ .Permalink }}" title="{{ safeHTML (i18n "commons.more_aria" (dict "Title" $title)) }}">{{ partial "PrepareHTML" $title }}</a>
-  <p class="meta">
+  <p class="publication-meta">
     {{ .Date.Format "2006" -}}{{- if .Params.authors -}},{{- end }} {{ .Params.authors }}
   </p>
+  <p class="publication-ref">
+    {{ partial "PrepareHTML" .Params.ref }}
+  </p>
 </article>