Fix #67: Create button crash — guard missing DOM element #70

Merged

SaschaFuksa merged 2 commits from batch/67 into master

2026-04-22 16:52:55 +00:00

Tasch commented

2026-04-22 16:22:48 +00:00

Owner

No description provided.

Tasch added 1 commit

2026-04-22 16:22:48 +00:00

Fix #67 : Create button crash — guard missing DOM element

CI / lint (pull_request) Waiting to run

Details

CI / test (pull_request) Waiting to run

Details

dc15637981

The structured panel was removed from the DOM during a UI redesign,
but loadStructured() still unconditionally calls getElementById('structuredList').
Add an early return guard if the element is not found.

Also refresh loadCurrentSession() after creating a structured transcript
instead of the removed loadStructured(), since the structured panel lives
in the current-session panel.

Tasch self-assigned this

2026-04-22 16:22:52 +00:00

Tasch added 1 commit

2026-04-22 16:23:57 +00:00

Fix #67 : add missing structured panel HTML and create button fallback

CI / lint (pull_request) Has been cancelled

Details

CI / test (pull_request) Has been cancelled

Details

2d26f73935

Two bugs fixed:

1. Structured panel HTML was missing from the page body.
   loadStructured() and createStructured() reference
   document.getElementById('structuredList') and
   document.getElementById('structuredReloadBtn'), but the
   structured-panel div with those IDs was never added to the
   DOM. This caused 'Cannot set properties of null (innerHTML)'
   when clicking Create.

2. createStructured() only searched for buttons inside
   .structured-item elements. The Create button rendered in
   session-item (.session-btn.structured-create) was never
   found, so the in-progress feedback never worked and the
   fallback alert firing after the API call was confusing.

Fixes:
- Added missing structured-panel HTML to the page body
  (matching the CSS already in the stylesheet).
- Added a fallback selector to find .session-item
  button.structured-create when no .structured-item button
  matches.
- Added transcript.txt write in chunk_transcription_mode so
  stopSession persists the merged transcript to the folder.