Fix missing ARIA attributes on dynamically inserted chapter rows

[Copilot]

Dynamically inserted chapter rows (via "Add Before"/"Add After") were missing role="textbox", aria-label on editable cells, and aria-label on all action buttons — attributes present on template-rendered rows — leaving screen readers unable to identify cell purpose or button actions.

Changes

• static/js/script.js
  • Extracted buildNewChapterRowHtml() helper, replacing the duplicated inline HTML strings in both btn-add-before and btn-add-after handlers
  • New rows now include full ARIA coverage matching the existing addChapterRow() pattern:
    • role="textbox" + aria-label on title and summary editable cells
    • aria-label on each button (move-up, move-down, add-before, add-after, delete-chapter)
    • aria-label on button group <div> elements

"<div class='editable-cell' contenteditable='true' data-field='title' role='textbox' aria-label='Chapter title'>New Chapter</div>"
// ...
"<button class='btn btn-sm btn-outline-danger btn-delete-chapter' title='Delete Chapter' aria-label='Delete chapter'>…</button>"

Generic labels (e.g. "Chapter title" rather than "Chapter 3 title") are used since renumberChapters() only updates visible text, not aria-label values.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• api.openai.com
  • Triggering command: /usr/bin/python python -m pytest tests/ -x -q --tb=short (dns block)
  • Triggering command: /usr/bin/python python -m pytest tests/ -q --tb=short (dns block)
  • Triggering command: /usr/bin/python python -m pytest tests/ --tb=short (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[Copilot]

Pull request overview

Fixes missing accessibility attributes on chapter rows that are inserted dynamically via “Add Before” / “Add After”, so screen readers can identify editable cells and action buttons consistently.

Changes:

• Added buildNewChapterRowHtml() helper to centralize new-row markup for both insertion paths.
• Ensured dynamically inserted rows include role="textbox" and aria-label on editable cells, plus aria-label on action buttons and button groups.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

buildNewChapterRowHtml() duplicates most of the markup already constructed in addChapterRow(), but with different escaping and ARIA-label conventions. To avoid the two templates drifting again (especially for accessibility attributes), consider extracting a single chapter-row builder that both addChapterRow() and the add-before/add-after handlers can use (e.g., parameterized by title/summary and optionally number).

[Copilot]

The new rows use generic aria-label values (e.g., "Chapter title"), while rows created via addChapterRow() use numbered labels (e.g., "Chapter 3 title"). Since renumberChapters() only updates the visible number, moving/inserting chapters can leave the existing numbered aria-labels incorrect. Consider updating renumberChapters() (or adding a helper it calls) to also recompute all chapter-related ARIA labels (cells, button groups, and buttons) based on the current index so screen readers get accurate, consistent labels.