Skip to content

fix(links): allow to pass custom link handler into editor and use it#8571

Open
mejo- wants to merge 3 commits into
mainfrom
fix/link_handler
Open

fix(links): allow to pass custom link handler into editor and use it#8571
mejo- wants to merge 3 commits into
mainfrom
fix/link_handler

Conversation

@mejo-
Copy link
Copy Markdown
Member

@mejo- mejo- commented May 12, 2026

📝 Summary

The return of the link handler 👻

Required as we want to pass a link handler from Collectives to be used for the "Open link" button and when opening a link via Ctrl-click.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits

@mejo- mejo- self-assigned this May 12, 2026
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team May 12, 2026
@mejo- mejo- moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team May 12, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 2026

Codecov Report

❌ Patch coverage is 60.71429% with 11 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/plugins/links.ts 60.00% 8 Missing and 2 partials ⚠️
src/components/Editor/PreviewOptions.vue 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@mejo- mejo- force-pushed the fix/link_handler branch from d04d70c to c1fe6d2 Compare May 12, 2026 09:22
Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the fix/link_handler branch from c1fe6d2 to 41455fa Compare May 12, 2026 11:22
mejo- added 2 commits May 12, 2026 13:46
The return of the link handler 👻

Required as we want to pass a link handler from Collectives to be used
for the "Open link" button and when opening a link via Ctrl-click.

Signed-off-by: Jonas <jonas@freesources.org>
Type majority of content while online. Should fix flakyness of the test
in CI.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the fix/link_handler branch from 41455fa to b99066b Compare May 12, 2026 11:49
mejo- added a commit to nextcloud/collectives that referenced this pull request May 12, 2026
Ensures that link navigation always uses the collectives link handler,
regardless of whether clicking the preview, the "Open link" button in
the link bubble, or Ctrl-clicking the link directly.

Requires nextcloud/text#8571

Fixes: #2378

Signed-off-by: Jonas <jonas@freesources.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

1 participant