Skip to content

XWIKI-24269: Improvements for the link insertion dialog#5395

Draft
ClementEXWiki wants to merge 1 commit intoxwiki:masterfrom
ClementEXWiki:feat/new-links-modal
Draft

XWIKI-24269: Improvements for the link insertion dialog#5395
ClementEXWiki wants to merge 1 commit intoxwiki:masterfrom
ClementEXWiki:feat/new-links-modal

Conversation

@ClementEXWiki
Copy link
Copy Markdown
Contributor

@ClementEXWiki ClementEXWiki commented Apr 20, 2026

Jira URL

https://jira.xwiki.org/browse/XWIKI-24269 (precursor to https://jira.xwiki.org/browse/XWIKI-23865)

Changes

Description

  • Improve the link insertion dialog

Clarifications

  • The link editor has now been rewritten in Vue (no more React)
  • The link editor is now a separate component, which can be used outside of a document editor
  • The link editor now follows the user's selected design system
  • Links can now be made to URLs, e-mail adresses (with prefilled message content), pages and attachments
  • Some more advanced features such as anchors are listed in the UI but not supported by BlockNote yet
  • Design is non-final, need to check that with Thiago
  • The suggestions list is implemented manually, as various DS' autocomplete components may have various UX which can be bothersome

Screenshots & Video

image

Executed Tests

N/A

Expected merging strategy

  • Prefers squash: Yes
  • Backport on branches:
    *

@ClementEXWiki ClementEXWiki marked this pull request as draft April 20, 2026 08:04
@socket-security
Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn Critical
Critical CVE: Prototype Pollution in npm lodash

CVE: GHSA-jf85-cpcp-j695 Prototype Pollution in lodash (CRITICAL)

Affected versions: < 4.17.12

Patched version: 4.17.12

From: xwiki-platform-core/xwiki-platform-bootstrap/src/main/package-lock.jsonnpm/grunt-jscs@3.0.1npm/lodash@4.6.1

ℹ Read more on: This package | This alert | What is a critical CVE?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/lodash@4.6.1. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant