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

Complete update for the newest Strata theme, with Jekyll enhancement. #13

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

fabacab
Copy link

@fabacab fabacab commented Jan 23, 2017

In the last two years, massive updates have been offered to the Strata
theme by HTML5 UP. This commit ports those new changes, including
several enhancements for Jekyll-specific use. For example:

  • Contact options configuration in the _config.yml file.
  • Dynamic copyright notices, also configurably in _config.yml.
  • Static assets are relative to the site.baseurl, now.
  • Contact form has been moved to an include for easier repitition.
  • Featured images and thumbnail images become post variables.
  • Option to add a static front page instead of most recent post.

@fabacab fabacab force-pushed the update-theme branch 2 times, most recently from 3895841 to bb6da0b Compare January 23, 2017 04:33
In the last two years, massive updates have been offered to the Strata
theme by HTML5 UP. This commit ports those new changes, including
several enhancements for Jekyll-specific use. For example:

* Contact options configuration in the `_config.yml` file.
* Dynamic copyright notices, also configurably in `_config.yml`.
* Static assets are relative to the `site.baseurl`, now.
* Contact form has been moved to an include for easier repitition.
* Featured images and thumbnail images become post variables.
* Option to add a static front page instead of most recent post.
@ghost
Copy link

ghost commented Jan 24, 2017

@meitar Thanks for the pull. I got to this a few days ahead of you with #12 which @mneumegen merged into master. Could you rebase your changes against the master branch of my strata-reloaded-jekyll-template fork after taking a look over the changes I made in #12. Please note @mneumegen mentioned in #12 CloudCannon and team are no longer maintaining this theme, and I can understand why given it's using a Creative Commons license as opposed to MIT.

@fabacab
Copy link
Author

fabacab commented Jan 24, 2017

Hi @JHabdas, no worries. I just merged your most recent changes into mine, so this should merge back cleanly now. I have prioritized making the HTML output match as close as possible to the HTML5 UP Strata demo's HTML, while not removing your additions (the "Edit" buttons and so on).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant