From c8a195775880526beb0b1bc62eec202a1b78df94 Mon Sep 17 00:00:00 2001
From: Segundo Fdez <segun.2@gmail.com>
Date: Wed, 19 Jan 2022 11:58:36 +0100
Subject: [PATCH] clean semantics and test svg on header

---
 ...s_5ad6f408b0e3e473c748aac88af0ea18.content | 15 ++++--
 src/themes/oscar/assets/css/main.css          | 11 ++++-
 .../oscar/layouts/_default/summary.html       | 12 ++---
 src/themes/oscar/layouts/index.html           |  6 +--
 src/themes/oscar/layouts/partials/header.html | 47 ++++++++++---------
 5 files changed, 53 insertions(+), 38 deletions(-)

diff --git a/src/resources/_gen/assets/css/css/main.css_5ad6f408b0e3e473c748aac88af0ea18.content b/src/resources/_gen/assets/css/css/main.css_5ad6f408b0e3e473c748aac88af0ea18.content
index deb2c47..2ca26b2 100644
--- a/src/resources/_gen/assets/css/css/main.css_5ad6f408b0e3e473c748aac88af0ea18.content
+++ b/src/resources/_gen/assets/css/css/main.css_5ad6f408b0e3e473c748aac88af0ea18.content
@@ -1,13 +1,20 @@
+@charset "UTF-8";
 /* Basic  */
-@import https://fonts.googleapis.com/css2?family=Overpass+Mono:wght@400;700&display=swap;
-@import url(/src/themes/oscar/assets/css/normalize.css);
-@import url(/src/themes/oscar/assets/css/variables.css);
-@import url(/src/themes/oscar/assets/css/common.css);
+/* @import 'https://fonts.googleapis.com/css2?family=Overpass+Mono:wght@400;700&display=swap';
+@import 'normalize.css';
+@import 'variables.css';
+@import 'common.css'; */
 /* Layout */
 /* Components */
 /* Modules */
 /* Specific pages */
 /* Utilities */
 /* Vendors */
+/*  TODO:
+    install postcss with:
+    - postcss-import
+    - postcss-custom-media
+    - postcss-nested (I think this is working by default, so probably we don´t need this plugin)
+ */
 
 /*# sourceMappingURL=style.css.map */
\ No newline at end of file
diff --git a/src/themes/oscar/assets/css/main.css b/src/themes/oscar/assets/css/main.css
index 08b56f2..c0676f7 100644
--- a/src/themes/oscar/assets/css/main.css
+++ b/src/themes/oscar/assets/css/main.css
@@ -1,8 +1,8 @@
 /* Basic  */
-@import 'https://fonts.googleapis.com/css2?family=Overpass+Mono:wght@400;700&display=swap';
+/* @import 'https://fonts.googleapis.com/css2?family=Overpass+Mono:wght@400;700&display=swap';
 @import 'normalize.css';
 @import 'variables.css';
-@import 'common.css';
+@import 'common.css'; */
 
 /* Layout */
 
@@ -15,3 +15,10 @@
 /* Utilities */
 
 /* Vendors */
+
+/*  TODO:
+    install postcss with:
+    - postcss-import
+    - postcss-custom-media
+    - postcss-nested (I think this is working by default, so probably we don´t need this plugin)
+ */
\ No newline at end of file
diff --git a/src/themes/oscar/layouts/_default/summary.html b/src/themes/oscar/layouts/_default/summary.html
index a673fe7..d5dce30 100644
--- a/src/themes/oscar/layouts/_default/summary.html
+++ b/src/themes/oscar/layouts/_default/summary.html
@@ -1,11 +1,9 @@
-<a href="{{ .Permalink }}" class="catalogue-item">
-    <div>
+<article>
+    <a href="{{ .Permalink }}" class="catalogue-item">
         <time datetime="{{ .PublishDate }}" class="catalogue-time">{{ i18n "publishdate" . }}</time>
-        <h2 class="catalogue-title">{{ .Title }}</h2>
-        <div class="catalogue-line"></div>
-
+        <h3>{{ .Title }}</h3>
         <p>
             {{ if isset .Params "summary" }}{{ .Params.Summary | markdownify }}{{ else }}{{ .Summary }}{{ end }}
         </p>
-    </div>
-</a>
+    </a>
+</article>
\ No newline at end of file
diff --git a/src/themes/oscar/layouts/index.html b/src/themes/oscar/layouts/index.html
index e497777..205d5a5 100644
--- a/src/themes/oscar/layouts/index.html
+++ b/src/themes/oscar/layouts/index.html
@@ -2,12 +2,12 @@
 
 <main>
 
-    <h1>List: Index</h1>
-	<div class="catalogue">
+	<section class="catalogue">
+		<h2>List: Index</h2>
 		{{ range (.Paginate (where .Site.RegularPages "Type" "posts")).Pages }}
 			{{ .Render "summary" }}
 		{{ end }}
-	</div>
+	</section>
 
 	<div class="pagination">
         {{ partial "pagination.html" . }}
diff --git a/src/themes/oscar/layouts/partials/header.html b/src/themes/oscar/layouts/partials/header.html
index 2c6bd33..2dd171b 100644
--- a/src/themes/oscar/layouts/partials/header.html
+++ b/src/themes/oscar/layouts/partials/header.html
@@ -1,23 +1,26 @@
-		<nav class="nav">
-			<div class="nav-container">
-				<a href="{{ "/" | relURL }}">
-					{{ if .IsPage }}
-						<h2 class="nav-title">{{ .Site.Title }}</h2>
-					{{ else }}
-						<h1 class="nav-title">{{ .Site.Title }}</h1>
-					{{ end }}
-				</a>
-                <ul>
-                    {{ $currentPage := . }}
-                    {{ range .Site.Menus.main }}
-                    <li>
-                        <a href="{{ .URL }}">
-                            {{ .Pre }}
-                            <span>{{ .Name }}</span>
-                            {{ .Post }}
-                        </a>
-                    </li>
-                    {{ end }}
-                </ul>
-			</div>
+	<header>
+        <h1>
+            <a href="{{ "/" | relURL }}">
+                <svg width="85" height="25" viewBox="0 0 85 25" fill="none" xmlns="http://www.w3.org/2000/svg">
+                    <rect x="28" width="26" height="25" fill="black"/>
+                    <path d="M60 0L85 25H60V0Z" fill="black"/>
+                    <rect width="26" height="25" rx="12.5" fill="black"/>
+                </svg>
+                <span>{{ .Site.Title }}</span>
+            </a>
+        </h1>
+        <nav>
+            <ul>
+                {{ $currentPage := . }}
+                {{ range .Site.Menus.main }}
+                <li>
+                    <a href="{{ .URL }}">
+                        {{ .Pre }}
+                        <span>{{ .Name }}</span>
+                        {{ .Post }}
+                    </a>
+                </li>
+                {{ end }}
+            </ul>
 		</nav>
+    </header>