-
Notifications
You must be signed in to change notification settings - Fork 575
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
fixes #59, baseurl to resolve // issue #71
Conversation
Applied the ruleset for baseurls outlined here http://downtothewire.io/2015/08/15/configuring-jekyll-for-user-and-project-github-pages/
Had to open new pull as last one was completely broken |
@lukemcgregor Sorry for the delay, I finally got around to reviewing these PRs and merging them. Thank you for your contributions. When you get a chance can you update this PR so we can get it in. Looks like the fix will make life easier for a lot of people. Cheers! |
@joshgerdes try now. Thanks for making this awesome skin 👍 If you are interested heres where I'm using it: http://blog.staticvoid.co.nz/2017/migrating-my-blog-to-jekyll-and-github-pages/ |
|
||
<link rel="stylesheet" href="{{ "css/main.css" | prepend: site.baseurl }}?{{site.time | date: '%s%N'}}"> | ||
<link rel="stylesheet" href="{{ site.baseurl }}/css/main.css?{{site.time | date: '%s%N'}}"> |
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.
@lukemcgregor : this line looks for css/main.css
which does not exist after this commit.
Applied the ruleset for baseurls outlined here
http://downtothewire.io/2015/08/15/configuring-jekyll-for-user-and-project-github-pages/