Skip to content
This repository was archived by the owner on Apr 28, 2019. It is now read-only.

fixing paths #189

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -189,14 +189,14 @@ position: 10

Getting closer, but no cigar. The rows are too close together, and I don't like that thumbnails in the same row aren't the same height.

Easy fix first: add this to `public/assets/styles.css`.
Easy fix first: add this to `public/stylesheets/styles.css`.

/* posts.liquid */
body.page-posts div.post-listing {
margin-bottom: 20px;
}

That solves the spacing, but what about the heights? Everyone knows that equalizing heights with CSS will cost you your youth, so I'm thinking JavaScript. There are plenty of plugins that do this, but it will probably be as simpler to implement it ourselves. Add the following two functions to the top of `public/assets/javascripts/main.js`.
That solves the spacing, but what about the heights? Everyone knows that equalizing heights with CSS will cost you your youth, so I'm thinking JavaScript. There are plenty of plugins that do this, but it will probably be as simpler to implement it ourselves. Add the following two functions to the top of `public/javascripts/main.js`.

function equalizeHeights(items) {
var maxHeight = 0;
Expand Down Expand Up @@ -251,4 +251,4 @@ position: 10

%a.orange-rounded-button{href: "/making-blog/3-3-pagination"} Next: pagination

{% endblock %}
{% endblock %}