From 15c0435dde9c1d42b248c3a3dd03b952e49add63 Mon Sep 17 00:00:00 2001
From: nipunrajk <nipunraj99@gmail.com>
Date: Sat, 26 Sep 2026 16:10:32 +0530
Subject: [PATCH] fix(markdown): prevent task lists with links from corrupting
 DOM

Disables label wrapping in markdown-it-task-lists plugin configuration.
When enabled, markdown-it-task-lists wraps task list content in <label>
tags, which malforms HTML structure when items contain links on the
same line and swallows subsequent DOM elements.

Fixes #715
---
 src/js/utils/markdown-editor.ts       |  2 +-
 src/tests/markdown-task-lists.test.ts | 65 +++++++++++++++++++++++++++
 2 files changed, 66 insertions(+), 1 deletion(-)
 create mode 100644 src/tests/markdown-task-lists.test.ts

diff --git a/src/js/utils/markdown-editor.ts b/src/js/utils/markdown-editor.ts
index a142d16f..5f603097 100644
--- a/src/js/utils/markdown-editor.ts
+++ b/src/js/utils/markdown-editor.ts
@@ -674,7 +674,7 @@ export class MarkdownEditor {
         .use(emoji) // Emoji: :smile: -> 😄
         .use(ins) // Inserted text: ++text++ -> <ins>text</ins>
         .use(mark) // Marked text: ==text== -> <mark>text</mark>
-        .use(taskLists, { enabled: true, label: true, labelAfter: true }) // Task lists: - [x] done
+        .use(taskLists, { enabled: true, label: false }) // Task lists: - [x] done
         .use(anchor, { permalink: false }) // Header anchors
         .use(tocDoneRight); // Table of contents: ${toc}
     }
diff --git a/src/tests/markdown-task-lists.test.ts b/src/tests/markdown-task-lists.test.ts
new file mode 100644
index 00000000..c0a105a8
--- /dev/null
+++ b/src/tests/markdown-task-lists.test.ts
@@ -0,0 +1,65 @@
+import { describe, it, expect, beforeEach } from 'vitest';
+import { MarkdownEditor } from '@/js/utils/markdown-editor';
+
+describe('Markdown to PDF - Task Lists with Links (Fixes #715)', () => {
+  let container: HTMLDivElement;
+
+  beforeEach(() => {
+    container = document.createElement('div');
+    document.body.appendChild(container);
+  });
+
+  it('renders task list items containing links without unclosed anchor tags or swallows', () => {
+    const markdown = `- [ ] [Links](https://bentopdf.com)
+
+Paragraph below link`;
+
+    new MarkdownEditor(container, {
+      initialContent: markdown,
+    });
+
+    const preview = container.querySelector('#mdPreview');
+    expect(preview).not.toBeNull();
+
+    const html = preview?.innerHTML || '';
+
+    // Check that the checkbox is rendered
+    expect(html).toContain('type="checkbox"');
+
+    // Check that the link is properly closed and contains the correct URL
+    const anchor = preview?.querySelector('a');
+    expect(anchor).not.toBeNull();
+    expect(anchor?.getAttribute('href')).toBe('https://bentopdf.com');
+    expect(anchor?.textContent?.trim()).toBe('Links');
+
+    // Check that the paragraph below is rendered as a separate sibling paragraph and NOT inside the anchor
+    const paragraph = preview?.querySelector('p');
+    expect(paragraph).not.toBeNull();
+    expect(paragraph?.textContent?.trim()).toBe('Paragraph below link');
+    expect(anchor?.contains(paragraph)).toBe(false);
+  });
+
+  it('correctly handles checked task lists with links and formatting', () => {
+    const markdown = `- [x] [Completed Link](https://bentopdf.com) with **bold text**
+- [ ] Regular task without link`;
+
+    new MarkdownEditor(container, {
+      initialContent: markdown,
+    });
+
+    const preview = container.querySelector('#mdPreview');
+    const checkboxes = preview?.querySelectorAll('input[type="checkbox"]');
+    expect(checkboxes?.length).toBe(2);
+
+    const checkedBox = checkboxes?.[0] as HTMLInputElement;
+    expect(checkedBox?.checked).toBe(true);
+
+    const anchor = preview?.querySelector('a');
+    expect(anchor?.getAttribute('href')).toBe('https://bentopdf.com');
+    expect(anchor?.textContent?.trim()).toBe('Completed Link');
+
+    const strong = preview?.querySelector('strong');
+    expect(strong?.textContent).toBe('bold text');
+    expect(anchor?.contains(strong)).toBe(false);
+  });
+});