Skip to content

fix: allow dots in markdown link text#8596

Open
SSDWGG wants to merge 1 commit into
nextcloud:mainfrom
SSDWGG:fix/markdown-link-text-dots
Open

fix: allow dots in markdown link text#8596
SSDWGG wants to merge 1 commit into
nextcloud:mainfrom
SSDWGG:fix/markdown-link-text-dots

Conversation

@SSDWGG
Copy link
Copy Markdown

@SSDWGG SSDWGG commented May 16, 2026

Summary

  • loosen the markdown link input rule so link text can contain dots
  • add a regression test for [something.cloud](https://something.cloud/page)

Fixes #7872

Tests

  • npx vitest run src/tests/marks/Link.spec.js

@mejo- mejo- force-pushed the fix/markdown-link-text-dots branch from 35b85e0 to 9bd164e Compare May 18, 2026 13:19
@mejo-
Copy link
Copy Markdown
Member

mejo- commented May 18, 2026

Thanks @SSDWGG, much appreciated. I slightly adjusted your commit and added a Playwright test to have end-to-end testing of the input rule.

@mejo- mejo- self-assigned this May 18, 2026
@mejo-
Copy link
Copy Markdown
Member

mejo- commented May 18, 2026

/backport to stable34

@mejo-
Copy link
Copy Markdown
Member

mejo- commented May 18, 2026

/backport to stable33

@codecov
Copy link
Copy Markdown

codecov Bot commented May 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the fix/markdown-link-text-dots branch from 9bd164e to ae3f8dd Compare May 18, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Creating links in Nextcloud Notes using md syntax does not always work.

2 participants