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

Remove all external dependencies #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

apankrat
Copy link

Hi Joseph,

I don't know if you might be interested in this patch, but just in case here it is. If not, then not. Just close the PR and no offense will be taken.

It removes external dependencies for MathJax and JQuery and strips the Google Analytics link. It also moves all secondary files into extras subfolder, just to make things a bit tidier Aside from this, there are no other changes.

Cheers,
Alex

@apankrat
Copy link
Author

Bah, forgot to say - Thanks a bunch for the code. Wonderful stuff, very clean.

I tweaked it a bit to my needs - reduced font sizes, tweaked keyboard shortcuts, extended markdown syntax a bit and hooked it up to simple server-based sync. Basically it now allows creating an offline-first webpage that can be edited at will -- makes for an excellent note taking app.

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