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 header and footer includes rather than repeating on every page! #6

Merged
merged 5 commits into from Jun 21, 2016

Conversation

@annashipman
Copy link
Owner

@annashipman annashipman commented Jun 21, 2016

This PR removes the default layout and moves the contents of that to header and footer includes, and passes as a parameter the name of the page to the header include, allowing it to be used for other pages rather than repeating the HTML on each page.

This PR also uses the new includes on JFDI, all blog posts, CV and Etc pages.

There are two pages left to handle:

  • index.html: my name is not a link (could be) and there is no colon
  • games.html: the head contains a meta-refresh tag to redirect.
annashipman added 5 commits Jun 20, 2016
Instead of having all the html in the default layout. This is a precursor to jekyll-ising the whole site, not just the blog.
This means it can be used for other pages.
No re-formatting of page, as more changes are afoot.
@annashipman annashipman merged commit 9238489 into master Jun 21, 2016
@annashipman annashipman deleted the use-header branch Jun 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant