Files
bitvid/todo/TODO_2026-02-18_15-05-00.md
google-labs-jules[bot]andPR0M3TH3AN 599f72a373 [security] replace innerHTML usages in js/ui/components/DeleteModal.js
- Replaced `wrapper.innerHTML` with `DOMParser` for template loading.
- Replaced `metadataList.innerHTML = ""` with `replaceChildren()`.
- Updated `scripts/check-innerhtml.mjs` baseline (count reduced by 3).
- Added artifacts: context, todo, decisions, log.
- Verified with lint and unit tests.

Co-authored-by: PR0M3TH3AN <[email protected]>
2026-02-18 16:23:29 +00:00

656 B

TODO - 2026-02-18

File: js/ui/components/DeleteModal.js

  1. Line 108: wrapper.innerHTML = html;
    • Context: Loading components/delete-video-modal.html template.
    • Strategy: Use DOMParser to parse the HTML string and append children to wrapper.
  2. Line 263: this.metadataList.innerHTML = "";
    • Context: Clearing the metadata list before re-rendering.
    • Strategy: Replace with this.metadataList.replaceChildren() or this.metadataList.textContent = "".
  3. Line 368: this.metadataList.innerHTML = "";
    • Context: Resetting the modal state.
    • Strategy: Replace with this.metadataList.replaceChildren().