New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update posts to move author to metadata #181
Conversation
@@ -0,0 +1,4 @@ | |||
<svg xmlns="http://www.w3.org/2000/svg" width="{{ include.width }}" height="{{ include.height }}" class="{{ include.class }}" viewBox="0 0 16 16" role="img" fill="currentColor"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we are not using different width/height, I'd say we better hardcode this. And also do this for the other icons assuming we don't pass any different values.
I'll check it out later to make sure the SEO tags are fine too. |
07190c7
to
ca9e47e
Compare
I pushed a few more patches. Not sure I love jekyll-avatar because of its lack of customization (even its lazy option doesn't work with our JS library nor can we pass And I'll try lazyloading the avatars later. |
ca9e47e
to
0f38d81
Compare
I reverted the jekyll-avatar addition and just moved common code to an include. Should be good to go. Looking good as a start! |
0f38d81
to
0d57778
Compare
Similarly to what we do with the videos.
We can expand this later to use `time`
27d8417
to
4a878c4
Compare
More traditional layout on index and individual posts by having author avatar and name at the top. Also adds an icon from Bootstrap Icons for the date, with a new date format that looks more complete.
Moves all post authors to single authors, to be in the front matter, and removes the
<3, ...
sign-off from all posts.