Refactor dartdoc markdown extensions into an ExtensionSet

[kevmoo]

Let's keep dartdoc and the pub site aligned here. No surprises or inconsistencies.

• Autolink w/out http[s] is kinda nice, but it's not consistent with how docs are rendered on GitHub or the package site.

• supporting [:...:] seems like a legacy holdover. We should remove these types of links from the SDK and then remove this, too.

[kevmoo]

CC @devoncarew

[devoncarew]

Where do we do the first one?

+1 to the 2nd item.

[kevmoo]

What (?) do we do the first one?

Render out http and https – as is standard.

If folks want pretty links, they can use the other syntax. Not worth the inconsistency IMHO

[devoncarew]

Ah, this was a flutter request - to display http://www.foo.bar as www.foo.bar and hyperlinked. It is prettier :)

[kevmoo]

Then at a minimum, we should create our own ExtensionSet in the markdown package and standardize on it.

At least so we're all together on our exceptions....

[jcollins-g]

I like the idea generally, but remember Dartdoc will always have extensions that are specific to it. It would be helpful if markdown's ExtensionSet had a public constructor and you could do something like:

new ExtensionSet.from(ExtensionSet.gitHub, inline: [new MoreStuff()], block: [new MoreStuffBlock()])

This would enable us to have a common base and add to it.

[kevmoo]

Open that issue!

…

On Tue, Dec 5, 2017 at 9:26 AM, jcollins-g ***@***.***> wrote: I like the idea generally, but remember Dartdoc will always have extensions that are specific to it. It would be helpful if markdown's ExtensionSet had a public constructor and you could do something like: new ExtensionSet.from(ExtensionSet.gitHub, inline: [new MoreStuff()], block: [new MoreStuffBlock()]) This would enable us to have a common base and add to it. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1534 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABCigS1wSQx-WR5i-xAf5GY0j0HsQqcks5s9XzYgaJpZM4QQLu1> .

[jcollins-g]

dart-lang/tools#1388