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

Replace plugins with includes for GHP compatibility #14

Merged

Conversation

mrmanc
Copy link
Collaborator

@mrmanc mrmanc commented Sep 3, 2018

The custom Jekyll plugins did not work with GitHub Pages so this change provides Jekyll includes which are compatible, although slightly more verbose in their use.

Fixes #7.

The custom Jekyll plugins did not work with GitHub Pages so this change
provides Jekyll includes which are compatible, although
slightly more verbose in their use.
@dploeger dploeger mentioned this pull request Sep 4, 2018
To accommodate a less verbose approach to marking items as fragments
and changing the background, the documentation mentions both approaches
with a note about the fact that using GitHub Pages requires the user
to use the include mechanism.
@dploeger
Copy link
Owner

dploeger commented Sep 5, 2018

Looks good. Could you please revert the sample slides to use the liquid tags, because we agreed on primarily supporting the liquid tags and only use the inclusion feature for GHP support.

I’ve moved the code from index.html into the layout since it now
contains some logic (the Liquid filter) which expands the syntax into
something that reveal.js understands.

Existing plugins are still supported.
@dploeger dploeger merged commit 2b1c66a into dploeger:master Sep 10, 2018
@mrmanc mrmanc deleted the fragments-and-background-includes branch September 10, 2018 17:36
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.

2 participants