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

A slimmed down fork - sending mainly as FYI :-) #134

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

Conversation

srinathh
Copy link

Hello - Thanks for releasing such a beautiful and functional theme with a permissive license. I bought you a few coffees :-) I have also forked your theme to remove dependencies on third party services that could be liabilities in today's legal environment or are in my opinion not suitable for simple static blogs on Github pages and also made a few fixes/updates. Sending along a PR in case you want to cherry pick any fixes.

Third Party Dependencies removed

  • Ad Sense
  • Disqus
  • Gravataar

Jekyll Plugins un-supportd by Github Pages removed

  • jekyll-toc
  • jekyll-archive

Other Changes

  • Loading JQuery from offical CDN vs. serving loacally
  • Removed lazy loading of styles & font
  • Removed Lunr search - client side search acoss lots of posts inefficient
  • Star Rating - not very useful without server backend
  • Outdated social sharing URLs fixed, Reddit sharing added
  • Added an Email button & added conditional check for Follow

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