Skip to content
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

Use jekyll for index and redirect pages as well #7

merged 9 commits into from Jun 22, 2016


Copy link

@annashipman annashipman commented Jun 21, 2016

This pull request:

  • uses the header and footer includes for index.html and the redirect layout
  • uses liquid to populate the index page with posts rather than handcrafted html
  • corrects some posts so that the excerpts appear correctly on the index page
annashipman added 9 commits Jun 21, 2016
This is to allow the home page to just be plain `Anna Shipman`.
This is now passed in as the page name, to allow the index page to not have a colon and extra spaces.
The index page doesn't need a page name. The only difference between this and what was there before is that the page heading is not a link to this page.
This requires passing two additional variables, the canonical link and the meta refresh. They can't be passed in inline and aren't available in the front matter, so this uses a liquid capture block to assign them to variables. It does mean there are blank lines in all the other pages, but that is acceptable.
Jekyll creates excerpts from all the text up to the `excerpt_separator`, which defaults to two newlines. This means that any posts that I wrote in html without blank lines between paragraphs in the source have as their exceprt the entire post.

This commit adds a blank line after first paragraph of the posts this affects. I don't intend to show more than 10 on the home page, but have done them all for consistency.
This changes some things; for example the excerpt is just the first paragraph whereas previously I wrote some excerpts differently. However, in general I write the first paragraph to be a trailer, and the ones that are not like this won't be on this page anyway as I intend to limit it.

It also means that the links are to the post and from there to whatever it is if it's elsewhere; previously a post about, e.g. a talk, would have the link to the video from here. I don't think this is too problematic.
There are no changes other than whitespace in this commit.
Some of the links open in a new page. Remove the ones on the index page that are still in use.

Also removes an extraneous full stop.
@annashipman annashipman merged commit ae9236a into master Jun 22, 2016
@annashipman annashipman deleted the deal-with-two-pages branch Jun 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant