-
Notifications
You must be signed in to change notification settings - Fork 97
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
Refactor code to modern standards #62
Comments
Hey @kelyvin, I would love to help! I literally just got my own ghost blog hosted yesterday and forked the uno-zen theme with the intention to use it. See https://nickjames.io. Last night I stumbled on your theme and will be using yours instead. I'm still getting the hang of ghost theme development, but I was able to make some enhancements to my fork of uno-zen. I removed bower and moved to npm, upgrade to gulp v4, added ES6 support using Babel, and removed some unnecessary dependencies. I will fork your theme to use for myself but will be opening some PRs soon to address some of the things outlined. Looking forward to helping out! |
@nickjamesio You are awesome! Please let me know if you have any questions or need any help along the way as well. Would also like to leverage ES6 modules as well instead of the spaghetti JQuery workarounds, so if you happen to switch to that as well, that would also be sweeeet |
Thanks! I am in the process of getting it setup on my local machine now. Had a few quirks to figure out since I am using docker but so far so good. I just successfully built the project using gulp so that is good :) I will definitely reach out if I get stuck or need help! |
I have sent a pull request for Font Awesome 5. I have tested it locally. |
It's been a long time since the code has been cleaned up. It originally started as a fork of another project but has since deviated drastically. As a result, it's appropriate to make updates to the internals. The refactor the code to leverage the following:
It's difficult trying to find the time to do this myself, so if anyone would like to volunteer, you'd be the real mvp.
The text was updated successfully, but these errors were encountered: