Replace inline links for template version #25152
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
As I wrote here, there are a ton of explicit html-type links in the docs, which makes
page.js
less effective in managing all these linksThis PR includes:
Replacing:
<a href="([\w\p{L}:/.\-_()?#=!~]+)" target="_blank">([\w\p{L}:/.\-_'\s]+)</a>
For:
[link:$1 $2]
And replacing:
<a href="([\w\p{L}:/.\-_()?#=!~]+)">([\w\p{L}:/.\-_'\s]+)</a>
For:
[link:$1 $2]
As I wrote in the above comment, I'm not sure what to do with the slightly more complicated variants like:
&
(example)(will require adjusting the regex in code.js)
rel=
(example)<a.*>\s*([\w\p{L}:/.\-_'\s]+)\s*</a>
and couldn't find anything else relevant - only internal links using id="")Testing URL
This contribution is funded by Resonai