X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/d20dd0c97eba396211fd546e241b0bdd557f9552..5ce1fd9f6cb8cf6243b92fef95b4705943ffef01:/doc/plugins/contrib/i18nheadinganchors/discussion.mdwn diff --git a/doc/plugins/contrib/i18nheadinganchors/discussion.mdwn b/doc/plugins/contrib/i18nheadinganchors/discussion.mdwn index ec6568010..7841467b2 100644 --- a/doc/plugins/contrib/i18nheadinganchors/discussion.mdwn +++ b/doc/plugins/contrib/i18nheadinganchors/discussion.mdwn @@ -69,6 +69,22 @@ supporting [IRIs](https://tools.ietf.org/html/rfc3987): ` which can easily be transliterated to "liberte". By using the +> RFC3987 scheme, we could use unicode directly in the links (`a +> href="#liberté"`), but the actual URL would be encoded as +> `#libert%e9`, which is really not as pretty. +> +> I understand you not wanting to introduce another dependency. And I +> also worry about the transliteration not being stable across +> releases. After all, it might not even be stable across Unicode +> releases either! But I'm ready to live with that inconvenience for +> the user-friendliness of the resulting URLs. --[[anarcat]] + ---- Documentation says: @@ -84,6 +100,10 @@ or similar. > It's not a bug, it's a limitation. :) But sure, it's a thing. It's an issue in > headinganchors as well of course. -- [[anarcat]] +>> No, current/historical headinganchors has a different bug: it ignores headings +>> that have any attributes, and does not generate anchors for them. That gives it +>> degraded functionality, but no information loss. I think that's less bad. --s + I think we should try to use an existing ID before generating our own, with the generation step as a fallback, just like Pandoc does. If a htmlize layer like Text::MultiMarkdown or Pandoc is generating worse IDs than this plugin, the @@ -98,6 +118,19 @@ htmlize layer, or stop using Text::MultiMarkdown. > for me to just override whatever attributes were there for testing and > fixing this in the short term... -- [[anarcat]] +> To bounce on this again: my problem with keeping existing IDs is +> that it basically makes headinganchors fail to do anything if +> something else adds the anchors. So I understand where you're coming +> from with this, but that "bug" was introduced on purpose, to +> actually fix a problem I was having. +> +> So I understand you might not want to *replace* headinganchors +> completely with this module, but could we at least merge it in so I +> wouldn't have to carry this patch around forever? :) Or what's our +> way forward here? +> +> Thanks! -- [[anarcat]] + ----
Some long scrollable text