Skip to content

feat: add link support to RichTextInput (PFR-1051)#3916

Open
RobD-tech wants to merge 1 commit into
edgefrom
feat/add-link-support-to-rich-text-input-PFR-1051
Open

feat: add link support to RichTextInput (PFR-1051)#3916
RobD-tech wants to merge 1 commit into
edgefrom
feat/add-link-support-to-rich-text-input-PFR-1051

Conversation

@RobD-tech

Copy link
Copy Markdown
Collaborator

Adds inline hyperlink functionality to the RichTextInput component:

  • Toolbar button (Ctrl/Cmd+K) opens an inline popover for inserting or editing a link without leaving the editor
  • URLs are auto-detected on paste and wrapped as links automatically
  • Existing links can be edited in-place; the popover pre-fills the current URL and open-in-new-tab state
  • "Open in new tab" checkbox adds target="_blank" rel="noreferrer" to the rendered anchor
  • Typing at the end of a link node moves the cursor outside the link so subsequent text is not included in the link
  • URL normalization: bare domains like "google.nl" are stored and serialized as "https://google.nl"
  • Popover labels (Save, Remove, Open in new tab) are rendered in the user's preferred browser language (EN, NL, DE, FR, ES, PT, with fallback EN)

Adds inline hyperlink functionality to the RichTextInput component:

- Toolbar button (Ctrl/Cmd+K) opens an inline popover for inserting or
  editing a link without leaving the editor
- URLs are auto-detected on paste and wrapped as links automatically
- Existing links can be edited in-place; the popover pre-fills the
  current URL and open-in-new-tab state
- "Open in new tab" checkbox adds target="_blank" rel="noreferrer" to
  the rendered anchor
- Typing at the end of a link node moves the cursor outside the link so
  subsequent text is not included in the link
- URL normalization: bare domains like "google.nl" are stored and
  serialized as "https://google.nl"
- Popover labels (Save, Remove, Open in new tab) are rendered in the
  user's preferred browser language (EN, NL, DE, FR, ES, PT)
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