Autolink should not include trailing punctuation

[dtdesign]

📝 Provide detailed reproduction steps (if any)

1. Visit the demo at https://ckeditor.com/ckeditor-5/demo/feature-rich/.
2. Copy the URL of the page into the clipboard.
3. Discard the editor contents.
4. Pretend that you are writing a sentence that ends with the URL from the clipboard.
5. Paste the URL from the clipboard at the end.
6. Hit . to end the sentence and hit <Space> to trigger the AutoLink plugin.

✔️ Expected result

The trailing dot should not be part of the implicitly created link.

❌ Actual result

The trailing dot is added at the end of the URL.

❓ Possible solution

The trailing space is already removed from the possible text in

ckeditor5/packages/ckeditor5-link/src/autolink.ts

Line 122 in 6d2257f

const url = getUrlAtTextEnd( text.substr( 0, text.length - 1 ) );

. This could be extended to check if the last character of url contains a punctuation symbol and remove it from the url value.

I thought about adding this to the regex, but it is already complex enough and it could possibly be faster to simply check the last character of url.

📃 Other details

• Browser: Chrome 114.0.5735.198
• OS: macOS 13.4.1
• First affected CKEditor version: Unknown
• Installed CKEditor plugins: Online Demo

---

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

[CKEditorBot]

There has been no activity on this issue for the past year. We've marked it as stale and will close it in 30 days. We understand it may still be relevant, so if you're interested in the solution, leave a comment or reaction under this issue.

[cadeyrn]

There has been no activity on this issue for the past year.

Not quite true… there is an open PR for this issue for a half year (!) and nobody (except a bot) reviewed it …