Permalink
Browse files

fix index post links

  • Loading branch information...
mdo committed Apr 19, 2015
1 parent e155800 commit 1c3cc17e0ef6f92f2a2b6d9e72c2192267f70f7c
Showing with 1 addition and 1 deletion.
  1. +1 −1 index.html
View
@@ -7,7 +7,7 @@
{% for post in paginator.posts %}
<div class="post">
<h1 class="post-title">
<a href="{{ site.baseurl }}{{ post.url }}">
<a href="{{ site.baseurl }}/{{ post.url }}">
{{ post.title }}
</a>
</h1>

4 comments on commit 1c3cc17

@mrneil

This comment has been minimized.

Show comment
Hide comment
@mrneil

mrneil Apr 26, 2015

Hi @mdo!

I'm pretty new at this Jekyll stuff and I'd like to learn why this change was made.

Using this version, I get two slashes at the beginning of the link. If I remove the slash so it's like the previous version, everything's correct (just a single slash in links).

It's also different than the same kind of link in _layouts/post.html.

Am I missing something really obvious? Is there a unique setup I need in order to see it?

Thanks for your work on Lanyon.

mrneil replied Apr 26, 2015

Hi @mdo!

I'm pretty new at this Jekyll stuff and I'd like to learn why this change was made.

Using this version, I get two slashes at the beginning of the link. If I remove the slash so it's like the previous version, everything's correct (just a single slash in links).

It's also different than the same kind of link in _layouts/post.html.

Am I missing something really obvious? Is there a unique setup I need in order to see it?

Thanks for your work on Lanyon.

@nickmccurdy

This comment has been minimized.

Show comment
Hide comment
@nickmccurdy

nickmccurdy Apr 27, 2015

Contributor

According to Jekyll's documentation for GitHub Pages, it's recommended to not have the extra slash and stick with {{ site.baseurl }}{{ post.url }}. Could someone please explain why this was changed?

Contributor

nickmccurdy replied Apr 27, 2015

According to Jekyll's documentation for GitHub Pages, it's recommended to not have the extra slash and stick with {{ site.baseurl }}{{ post.url }}. Could someone please explain why this was changed?

@contraultra

This comment has been minimized.

Show comment
Hide comment
@contraultra

contraultra Apr 30, 2015

I reverted this since this breaks the post link.

contraultra replied Apr 30, 2015

I reverted this since this breaks the post link.

@funkenstrahlen

This comment has been minimized.

Show comment
Hide comment
Please sign in to comment.