Skip to content

Commit

Permalink
Revert back plugins: and update references to site.gems with `si…
Browse files Browse the repository at this point in the history
…te.plugins`

- Conditionals that determine if jekyll-paginate and jekyll-feed are active weren't working due `gems` key change in Jekyll 3.5

ref mmistakes#24
  • Loading branch information
mmistakes committed Aug 9, 2017
1 parent 428b2b0 commit 1c571db
Show file tree
Hide file tree
Showing 9 changed files with 16 additions and 14 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/)
and this project adheres to [Semantic Versioning](http://semver.org/).

## [Unreleased]
### Changed
- Change `gems` key in `_config.yml` to `plugins`. [#23](https://github.com/mmistakes/jekyll-theme-basically-basic/pull/23)

## [1.1.0] - 2017-03-30
### Added
Expand Down
2 changes: 1 addition & 1 deletion _config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ logo: # path of site logo, e.g. "/assets/images/logo.png"
# Build settings
markdown: kramdown
theme: jekyll-theme-basically-basic
gems:
plugins:
- jekyll-feed
- jekyll-seo-tag
- jekyll-sitemap
Expand Down
4 changes: 2 additions & 2 deletions _includes/contact-list.html
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,11 @@
{% endif %}

<li>
{% if site.gems contains 'jekyll-feed' %}
{% if site.plugins contains 'jekyll-feed' %}
<a href="{{ site.feed.path | default: 'feed.xml' | relative_url }}" title="Atom Feed">
<span class="icon icon--rss">{% include icon-rss.svg %}</span>
<span class="label">{{ site.data.theme.t.subscribe | default: 'Subscribe' }}</span>
</a>
{% endif %}
</li>
</ul>
</ul>
2 changes: 1 addition & 1 deletion _layouts/home.html
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
<h2>{{ site.data.theme.t.posts | default: 'Posts' }}{% if paginator.page > 1 %}{{ site.data.theme.t.page | default: 'Page' | prepend: ' - ' | append: ' ' }}{{ paginator.page }} {{ site.data.theme.t.of | default: 'of' }} {{ paginator.total_pages }}{% endif %}</h2>
</header>
<div class="entries">
{% if site.gems contains 'jekyll-paginate' and page.paginate %}
{% if site.plugins contains 'jekyll-paginate' and page.paginate %}
{% comment %}
Add paginator.posts loop if jekyll-paginate plugin is enabled
and page.paginate == true
Expand Down
10 changes: 5 additions & 5 deletions docs/Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@ gem "wdm", "~> 0.1.0" if Gem.win_platform?

# If you have any plugins, put them here!
# group :jekyll_plugins do
# gem "jekyll-paginate"
# gem "jekyll-sitemap"
# gem "jekyll-gist"
# gem "jekyll-feed"
# gem "jemoji"
# gem "jekyll-paginate"
# gem "jekyll-sitemap"
# gem "jekyll-gist"
# gem "jekyll-feed"
# gem "jemoji"
# end
2 changes: 1 addition & 1 deletion docs/_config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ permalink: /:categories/:title/
timezone: America/New_York # http://en.wikipedia.org/wiki/List_of_tz_database_time_zones

# Plugins
gems:
plugins:
- jekyll-feed
- jekyll-seo-tag
- jekyll-sitemap
Expand Down
4 changes: 2 additions & 2 deletions docs/_includes/contact-list.html
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,11 @@
{% endif %}

<li>
{% if site.gems contains 'jekyll-feed' %}
{% if site.plugins contains 'jekyll-feed' %}
<a href="{{ site.feed.path | default: 'feed.xml' | relative_url }}" title="Atom Feed">
<span class="icon icon--rss">{% include icon-rss.svg %}</span>
<span class="label">{{ site.data.theme.t.subscribe | default: 'Subscribe' }}</span>
</a>
{% endif %}
</li>
</ul>
</ul>
2 changes: 1 addition & 1 deletion docs/_layouts/home.html
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
<h2>{{ site.data.theme.t.posts | default: 'Posts' }}{% if paginator.page > 1 %}{{ site.data.theme.t.page | default: 'Page' | prepend: ' - ' | append: ' ' }}{{ paginator.page }} {{ site.data.theme.t.of | default: 'of' }} {{ paginator.total_pages }}{% endif %}</h2>
</header>
<div class="entries">
{% if site.gems contains 'jekyll-paginate' and page.paginate %}
{% if site.plugins contains 'jekyll-paginate' and page.paginate %}
{% comment %}
Add paginator.posts loop if jekyll-paginate plugin is enabled
and page.paginate == true
Expand Down
2 changes: 1 addition & 1 deletion example/_config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ logo: /assets/icons/basically-basic-logo-light.svg
# Build settings
markdown: kramdown
theme: jekyll-theme-basically-basic
gems:
plugins:
- jekyll-feed
- jekyll-seo-tag
- jekyll-sitemap
Expand Down

0 comments on commit 1c571db

Please sign in to comment.