From 5cce3b726ae898732a258a137bca5ec8689a088c Mon Sep 17 00:00:00 2001 From: thePR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com> Date: Thu, 30 Jul 2026 11:45:23 -0400 Subject: [PATCH] =?UTF-8?q?docs:=20close=20stale=20innerHTML=20TODO=20?= =?UTF-8?q?=E2=80=94=20ShareNostrModal=20has=20no=20innerHTML=20left?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01LHH3wPBsoruwCaJo8Rpf6a --- todo/TODO_2026-02-20_innerhtml.md | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/todo/TODO_2026-02-20_innerhtml.md b/todo/TODO_2026-02-20_innerhtml.md index 239cd222..3b464d2b 100644 --- a/todo/TODO_2026-02-20_innerhtml.md +++ b/todo/TODO_2026-02-20_innerhtml.md @@ -1,8 +1,14 @@ # TODO: innerHTML Migration for ShareNostrModal.js -- [ ] L107: `wrapper.innerHTML = html;` (Template loading) +- [x] L107: `wrapper.innerHTML = html;` (Template loading) - Strategy: Use `new DOMParser().parseFromString(html, 'text/html')` and append children to `wrapper`. -- [ ] L274: `this.relayPills.innerHTML = "";` (Clearing content) +- [x] L274: `this.relayPills.innerHTML = "";` (Clearing content) - Strategy: `this.relayPills.replaceChildren();` or `textContent = ""` (replaceChildren is preferred for performance). -- [ ] L300: `removeButton.innerHTML = '...';` (SVG Icon) +- [x] L300: `removeButton.innerHTML = '...';` (SVG Icon) - Strategy: Create SVG elements using `document.createElementNS("http://www.w3.org/2000/svg", "svg")` and children. + +--- + +**CLOSED 2026-07-30.** All three migrations already landed in later refactors — +`grep innerHTML js/ui/components/ShareNostrModal.js` returns nothing and the +innerHTML lint baseline holds. Kept for the record.