Could not linkify the URL with super-long TLDs.

[tmyt]

Currently Linkifier implimentation could not linkify the URL with super-long TLDs.

Linkifier assume the TLD lenght in 2 - 6 chars in the here,
https://github.com/sourcelair/xterm.js/blob/master/src/Linkifier.ts#L14

But, some new gTLDs have over 6 chars. (ex: .enterprises, .foundation, and so on).
Needs to update tldClause to support these super-long TLD names.

Details

• xterm.js version: master (ff29fd6)

Steps to reproduce

1. echo 'https://example.enterprises'
2. URL does not linked.

[Tyriar]

PRs welcome, should be an easy fix 👍

[jens1o]

@Tyriar Would happy to contribute, for me 256 ought to be enough for anybody? ;)

[Tyriar]

@jens1o the concern with working more on the web link matcher as is, is that it will eventually be replaced by something much smarter that can understand things like when brackets () are within or outside of a URL, see #583

And that one's a bit more work 😛 as it's replacing all the regex logic with a parser that will probably work by finding the start of a link and then move forward until the end is found.

[jens1o]

Oh, sorry. I think I won't meet your expectations... Let's hope there will be some nice contributor with some good skills :D

[Tyriar]

No worries 😄

[Tyriar]

Merging into #583