fix: clipboard helper HTML priority

Author: distantnative

panel/src/helpers/clipboard.test.ts

@@ -55,6 +55,25 @@ describe.concurrent("clipboard.read()", () => {
 		expect(read(event, true)).toBe("hello");
 	});
 
+	it("should prefer text/html over text/plain", () => {
+		const dataTransfer = new MockDataTransfer();
+		dataTransfer.setData("text/html", "<b>hello</b>");
+		dataTransfer.setData("text/plain", "hello");
+		const event = new MockClipboardEvent("paste", {
+			clipboardData: dataTransfer
+		});
+		expect(read(event)).toBe("<b>hello</b>");
+	});
+
+	it("should fall back to text/plain when text/html is absent", () => {
+		const dataTransfer = new MockDataTransfer();
+		dataTransfer.setData("text/plain", "hello");
+		const event = new MockClipboardEvent("paste", {
+			clipboardData: dataTransfer
+		});
+		expect(read(event)).toBe("hello");
+	});
+
 	it("should replace non-breaking spaces", () => {
 		const dataTransfer = new MockDataTransfer();
 		dataTransfer.setData("text/plain", "hello\u00a0world");

panel/src/helpers/clipboard.ts

@@ -13,13 +13,15 @@ export function read(
 	if (e instanceof ClipboardEvent) {
 		e.preventDefault();
 
-		const text = e.clipboardData?.getData("text/plain") ?? null;
+		// getData() returns "" for absent types,
+		// use || not ?? to treat empty strings as missing
+		const text = e.clipboardData?.getData("text/plain") || null;
 
 		if (plain === true) {
 			return text;
 		}
 
-		const html = text ?? e.clipboardData?.getData("text/plain") ?? null;
+		const html = e.clipboardData?.getData("text/html") || text || null;
 
 		if (html) {
 			return html.replace(/\u00a0/g, " ");