Skip to content

Commit 2ff1c5c

Browse files
Merge pull request #280 from DiogoRibeiro7/codex/update-styles-and-verify-appearance
Apply design tweaks
2 parents 9f96279 + af77162 commit 2ff1c5c

File tree

4 files changed

+47
-5
lines changed

4 files changed

+47
-5
lines changed

README.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,18 @@ GitHub Actions already runs these commands automatically during deployments.
7373

7474
# ToDo
7575

76+
~~Have a consistency in the font and font sizes (ideally you want to use 2 fonts. One for the header/subtitle and one for the text. You can use this kind of website https://fontjoy.com/ which allow you to pair fonts).~~
77+
7678
Choose a few main colours for your site (I would suggest black/white/grey but not in solid. You can also use this kind of site: https://coolors.co/palettes/popular/2a4849).
7779

80+
~~Reduce then size of the homepage top image (ideally you want your first articles to be visible on load and not hidden below the fold).~~
81+
82+
~~Restyle your links (ideally the link should be back with no underline and you add a css style on hover)~~
83+
84+
~~Center pagination~~
85+
86+
~~Restyle your article detail page breadcrumbs. You want them to be less visible (I would suggest a light grey colour here)~~
87+
7888
Right now at the top of the detail page, you have your site breadcrumbs, a title then another title and the font sizes are a bit off and it is hard to understand the role of the second title. I would reorganise this to provide a better understanding to the reader
7989
On the detail page, I would suggest you put the `You may also enjoy` straight at the end of the article. Right now it is after comments and you can lose engagement on your site.
8090
I would suggest you remove your description from the detail page. I think having it on the home page is enough. You can have a smaller introduction if needed with a read more button or link that will take the reader to a full page description of yourself and your skillset. That will allow you to tell more about yourself and why you do what you do

_includes/head.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@
2020
<!-- Google Fonts -->
2121
<link rel="preconnect" href="https://fonts.googleapis.com">
2222
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
23-
<link href="https://fonts.googleapis.com/css2?family=Open+Sans:wght@400;700&family=Montserrat:wght@700&display=swap" rel="stylesheet">
23+
<link href="https://fonts.googleapis.com/css2?family=Cardo&display=swap" rel="stylesheet">
24+
<link href="https://fonts.googleapis.com/css2?family=Merriweather:wght@400;700&family=Open+Sans:wght@400;700&display=swap" rel="stylesheet">
2425

2526
{% if site.head_scripts %}
2627
{% for script in site.head_scripts %}

_sass/minimal-mistakes/_variables.scss

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
Typography
77
========================================================================== */
88

9-
// Cardo font is loaded via a <link> tag in the site head
9+
// Cardo and custom web fonts are loaded via <link> tags in the site head
1010

1111
$doc-font-size: 16 !default;
1212

@@ -32,9 +32,9 @@ $calisto: "Calisto MT", serif !default;
3232
$garamond: Garamond, serif !default;
3333

3434
// Setting the fonts
35-
$global-font-family: "Open Sans", sans-serif;
36-
$header-font-family: "Montserrat", sans-serif;
37-
$caption-font-family: "Open Sans", sans-serif;
35+
$global-font-family: "Open Sans", Helvetica, Arial, sans-serif;
36+
$header-font-family: "Merriweather", "Times New Roman", serif;
37+
$caption-font-family: "Cardo, serif";
3838

3939
/* type scale */
4040
$type-size-1: 2.441em !default; // ~39.056px

assets/css/main.scss

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -103,3 +103,34 @@ a:hover {
103103
.page__hero-button {
104104
margin-top: 1px; /* Add some space between the content and the button */
105105
}
106+
107+
/* Breadcrumbs in light grey */
108+
.breadcrumbs,
109+
.breadcrumbs a {
110+
color: $light-gray;
111+
}
112+
113+
/* Center pagination controls */
114+
.pagination {
115+
text-align: center;
116+
}
117+
118+
.pagination ul {
119+
display: inline-block;
120+
padding-left: 0;
121+
}
122+
123+
.pagination li {
124+
float: none;
125+
display: inline-block;
126+
}
127+
128+
/* Smaller hero section on homepage */
129+
.layout--home .page__hero--overlay {
130+
padding: 2em 0;
131+
}
132+
133+
.layout--home .page__hero-image img {
134+
max-height: 40vh;
135+
object-fit: cover;
136+
}

0 commit comments

Comments
 (0)