Add heading-anchor extension

[mcruzdev]

This pull request adds support for heading anchor!

It is a great feature for feature features, like searching...

[mkouba]

Does it make sense to introduce a new extension for this functionality? It could be a functionality of the core markdown extension, probably guarded by a config flag...

[mcruzdev]

Does it make sense to introduce a new extension for this functionality? It could be a functionality of the core markdown extension, probably guarded by a config flag...

I really prefer this approach for the simplicity, it would be enabled by default.

[ia3andy]

@mkouba I am also not sure about how to deal with those, but since they are runtime dependencies, better split like this IMO

[mkouba]

@mkouba I am also not sure about how to deal with those, but since they are runtime dependencies, better split like this IMO

Does it make sense to introduce a new extension for this functionality? It could be a functionality of the core markdown extension, probably guarded by a config flag...

I really prefer this approach for the simplicity, it would be enabled by default.

@mcruzdev @ia3andy I don't think it's reasonable to add a new extension for each major feature. IMO a new extension is adequate if a feature requires a new dependency (another extension or library).

[ia3andy]

It requires a new dependency

[mkouba]

It requires a new dependency

Ah, sorry I completely missed the commonmark-ext-heading-anchor dependency. So pls ignore my previous comments ;-).

[mcruzdev]

@ia3andy we need to add this feature for asciidoc too. WDYT to add this feature on roq? manipulating the HTML and adding heading IDs? To add a mechanism to change the HTML through plugins too.

[mcruzdev]

Hi @mkouba, do you think when you throw the new release?

[mkouba]

Hi @mkouba, do you think when you throw the new release?

Once we fine tune and merge #129 ;-).