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

Added txtpen in tutorial #6021

Merged
merged 5 commits into from Apr 13, 2017

Conversation

Projects
None yet
5 participants
@rickyhan
Contributor

rickyhan commented Apr 11, 2017

txtpen is the only service that provides highlighting and in-text commenting similar to Medium. I think adding txtpen in the official documentation will benefit new Jekyll users and bring web annotation to more people.

Thanks in advance.
Ricky

rickyhan added some commits Apr 10, 2017

@pathawks

Extras might be a better place for this, along with MathJax support.

Instead of just linking to homepage, it would probably be better to link to specific instructions for integrating the service into a Jekyll site.

@rickyhan

This comment has been minimized.

Show comment
Hide comment
@rickyhan

rickyhan Apr 12, 2017

Contributor

I have moved the txtpen installation instruction to extras.

Thank you very much 😄

Contributor

rickyhan commented Apr 12, 2017

I have moved the txtpen installation instruction to extras.

Thank you very much 😄

Show outdated Hide outdated docs/_docs/extras.md
Register your site with [txtpen](https://txtpen.com). Then append
<script src="https://txtpen.com/embed.js?site=<your site name>" />

This comment has been minimized.

@oe

oe Apr 12, 2017

Member

could you use fenced code blocks? you can find an example in the Math Support section just below this one

@oe

oe Apr 12, 2017

Member

could you use fenced code blocks? you can find an example in the Math Support section just below this one

This comment has been minimized.

@rickyhan

rickyhan Apr 12, 2017

Contributor

Done 👍

@rickyhan

rickyhan Apr 12, 2017

Contributor

Done 👍

@rickyhan

This comment has been minimized.

Show comment
Hide comment
@rickyhan

rickyhan Apr 13, 2017

Contributor

@pnn @pathawks

Following up on this. What else needs to be done?

Contributor

rickyhan commented Apr 13, 2017

@pnn @pathawks

Following up on this. What else needs to be done?

@pathawks

This comment has been minimized.

Show comment
Hide comment
Member

pathawks commented Apr 13, 2017

@pathawks

This comment has been minimized.

Show comment
Hide comment
@pathawks

pathawks Apr 13, 2017

Member

@rickyhan I guess the main thing is that we need to decide if this is something that would be useful to a large number of our users. Are there a lot of Jekyll sites using fhis service? Are there resources online to help users integrate the service into their Jekyll site? Are any Jekyll maintainers currently using the service?

Member

pathawks commented Apr 13, 2017

@rickyhan I guess the main thing is that we need to decide if this is something that would be useful to a large number of our users. Are there a lot of Jekyll sites using fhis service? Are there resources online to help users integrate the service into their Jekyll site? Are any Jekyll maintainers currently using the service?

@DirtyF

DirtyF approved these changes Apr 13, 2017 edited

Unless I'm mistaken this is a brand new service and this PR only aims at giving it some visibility. As it complies with a W3C standard, I have no objection about adding this in extra.

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Apr 13, 2017

Member

@jekyllbot: merge +docs

Member

DirtyF commented Apr 13, 2017

@jekyllbot: merge +docs

@jekyllbot jekyllbot merged commit 87dd496 into jekyll:master Apr 13, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

jekyllbot added a commit that referenced this pull request Apr 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment