P2: Task Note root hiding removes nested checklist and paragraph content #935

Open
opened 2026-10-02 17:41:22 +00:00 by kayg · 0 comments
Owner

P2 — Hiding the root also hides its child content

Evidence: packages/editor/src/components/TaskItemView.svelte:56 applies
display: none to the whole root li. Its NodeViewContent at line 52 holds
both the title paragraph and nested blocks. A Task Note with an indented
checklist or paragraph under its root therefore loses that content from the
visible editor and the accessibility tree. The module comment says children
stay visible, but the CSS hides their ancestor.

Fix: hide only the root checkbox and title paragraph. Keep nested block content
visible and editable. Mark only the redundant title controls as hidden from
screen readers.

Test idea: open - [ ] Parent with an indented child checkbox and a paragraph.
Assert that the parent title occurs once, the child checkbox can be ticked,
and the paragraph can be edited and survives reload.

Rule: DESIGN §9 and the owner accessibility and UX completeness rules.

Source: independent read-only review of #659 at 013f6785a. Line numbers refer to that commit. No build or runtime test was run.

## P2 — Hiding the root also hides its child content Evidence: `packages/editor/src/components/TaskItemView.svelte:56` applies `display: none` to the whole root `li`. Its `NodeViewContent` at line 52 holds both the title paragraph and nested blocks. A Task Note with an indented checklist or paragraph under its root therefore loses that content from the visible editor and the accessibility tree. The module comment says children stay visible, but the CSS hides their ancestor. Fix: hide only the root checkbox and title paragraph. Keep nested block content visible and editable. Mark only the redundant title controls as hidden from screen readers. Test idea: open `- [ ] Parent` with an indented child checkbox and a paragraph. Assert that the parent title occurs once, the child checkbox can be ticked, and the paragraph can be edited and survives reload. Rule: DESIGN §9 and the owner accessibility and UX completeness rules. Source: independent read-only review of #659 at 013f6785a. Line numbers refer to that commit. No build or runtime test was run.
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kayg/calternal#935
No description provided.