Add automatic hyperlinks

[anseljh]

Adding links to the HTML was way easier than the .DOCX, so I tacked it first.

[kemitchell]

Thanks so much for this!

Linkifying HTML is a common enough general problem that I've actually been doing it with an external script:

// linkify.js
var linkify = require('html-linkify')
var input = require('fs').readFileSync('/dev/stdin').toString()
process.stdout.write(linkify(input, { escape: false }))

Then:

$ commonform render -f html < agreement.commonform | node linkify.js > agreement.html

Incidentally, linkify handles e-mail addresses. It's a nice, tight package.

Three questions in my mind:

1. Might users of commonform-html ever not want linkification?
2. If so, should the functionality go behind an options-argument flag?
3. What's the case for leaving linkification out of commonform-html, and perhaps mentioning a package like linkify in README or other doc?

Again, thanks so much. I will have a think today or tomorrow. Would love to read your thoughts, too!

[anseljh]

I can't think of a reason not to linkify HTML.

Maybe a better approach is to use that library as a post-processing step at the end of the HTML rendering.

[anseljh]

I've squinted at this for a while, but don't yet understand why the test fails. 😠

Instead of:
<div class="article"><p>This is a great website: <a href="http://example.com/">http://example.com/</a></p></div>

it produces:
<div class="article"><p>This is a great website: <a href="<a href="http://example.com/">http://example.com/</a>"><a href="http://example.com/">http://example.com/</a></a></p></div>