External/buzz
1
0
Fork 0
mirror of https://github.com/block/buzz.git synced 2026-08-18 06:50:31 +02:00
Code Issues Packages Projects Releases Wiki Activity

docs(composer): trim verbose buzz:// paste comments

Browse Source 
Comment-only pass — logic unchanged. Keeps the why (linkifyjs doesn't
know buzz://) and drops the play-by-play.

Co-authored-by: Taylor Ho <taylorkmho@gmail.com>
Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
This commit is contained in:
npub1223z34hd7vtwc6qj4s7flsxkj644nlre2nthu7lrrmkumhu3xddsrx9r6w
2026-06-14 11:16:11 -07:00
co-authored by Taylor Ho
parent 1140a4c77f
commit c1d43210ff
2 changed files with 6 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
desktop/src/features/messages/lib/messageLink.ts
+2 -7
View File
@@ -104,13 +104,8 @@ export function isMessageLink(href: string | undefined | null): boolean {
}
/**
* Returns true if `text` is any `buzz://` URL (not just the message-link
* format). Used by the composer's wrap-on-paste handler: linkifyjs (which
* TipTap's built-in paste handler relies on) doesn't recognise the `buzz`
* scheme, so we detect it ourselves and wrap the highlighted selection.
*
* Validated via `URL` so we only treat genuinely well-formed `buzz://` URLs as
* links — a bare `buzz:` or malformed string won't match.
* Returns true if `text` is any well-formed `buzz://` URL (not just the
* message-link format) — used by the composer's wrap-on-paste handler.
*/
export function isBuzzUrl(text: string | undefined | null): boolean {
if (!text) return false;
desktop/src/features/messages/lib/useRichTextEditor.ts
+4 -11
View File
@@ -306,13 +306,9 @@ export function useRichTextEditor({
inclusive() {
return false;
},
// The built-in `linkOnPaste` handler detects URLs with linkifyjs,
// which only knows standard schemes (http/https/mailto) — so pasting
// a `buzz://` URL over a text selection clobbers it with raw text
// instead of wrapping it in a link. Prepend our own paste handler
// that recognises `buzz://` URLs and wraps the highlighted selection,
// matching the standard-URL UX. Runs before the parent plugins so it
// claims the paste first; otherwise we defer to TipTap's defaults.
// linkifyjs (which TipTap's `linkOnPaste` uses) doesn't recognise
// `buzz://`, so wrap such links ourselves. Runs before the parent
// plugins; defers to TipTap's defaults otherwise.
addProseMirrorPlugins() {
const linkType = this.type;
const buzzLinkPaste = new Plugin({
@@ -320,8 +316,6 @@ export function useRichTextEditor({
props: {
handlePaste: (view, _event, slice) => {
const { selection } = view.state;
// Only wrap when there's a selection to wrap — empty
// selections fall through to TipTap's normal insert path.
if (selection.empty) return false;
let textContent = "";
@@ -329,9 +323,8 @@ export function useRichTextEditor({
textContent += node.textContent;
});
// Only wrap when the whole payload is a single buzz:// URL.
const href = textContent.trim();
// The whole clipboard payload must be a single buzz:// URL —
// same contract as the built-in handler (`value === text`).
if (!isBuzzUrl(href)) return false;
return this.editor