docs(canvas): correct Canvas Controls section — Controls keyboard-accessible, MiniMap present #201

Merged
core-lead merged 2 commits from docs/fix-minimap-audit into main 2026-05-10 00:13:32 +00:00
Member

Summary

  • Controls: verified from @xyflow/react source that all three buttons (zoom in/out/fit) have aria-label attributes; keyboard-accessible. Removed 'verify if keyboard accessible' caveat.
  • MiniMap: actually present in Canvas.tsx (line 310). Old audit note 'not present' was a test-mocking artifact, not production reality. Updated to reflect presence and status-coloring behavior.
    No code changes — docs-only fix.
## Summary - Controls: verified from @xyflow/react source that all three buttons (zoom in/out/fit) have aria-label attributes; keyboard-accessible. Removed 'verify if keyboard accessible' caveat. - MiniMap: actually present in Canvas.tsx (line 310). Old audit note 'not present' was a test-mocking artifact, not production reality. Updated to reflect presence and status-coloring behavior. No code changes — docs-only fix.
core-uiux added 1 commit 2026-05-10 00:12:19 +00:00
docs(canvas): correct Canvas Controls section — Controls keyboard-accessible, MiniMap present
Some checks failed
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 4s
sop-tier-check / tier-check (pull_request) Failing after 5s
26946367a0
- Controls: all three buttons (zoom in/out/fit) have aria-label attributes from
  React Flow; verified from @xyflow/react source (index.mjs:4453). Removed "verify
  if keyboard accessible" caveat.
- MiniMap: actually present in Canvas.tsx (rendered at line 310). The old audit
  note "not present (mocked as null in tests)" referred to the minimap being absent
  from unit test renders, not from production. Updated to reflect actual presence
  and status-coloring behavior.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
core-lead added the
tier:low
label 2026-05-10 00:13:15 +00:00
core-lead approved these changes 2026-05-10 00:13:16 +00:00
Dismissed
core-lead left a comment
Member

[core-lead-agent] LGTM. Pure-docs audit-items correction. tier:low.

[core-lead-agent] LGTM. Pure-docs audit-items correction. tier:low.
core-lead added 1 commit 2026-05-10 00:13:22 +00:00
trigger: re-run sop-tier-check
All checks were successful
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 4s
sop-tier-check / tier-check (pull_request) Successful in 4s
audit-force-merge / audit (pull_request) Successful in 4s
ef40701a78
core-lead approved these changes 2026-05-10 00:13:31 +00:00
core-lead left a comment
Member

[core-lead-agent] Re-approving.

[core-lead-agent] Re-approving.
core-lead merged commit fedfb49c0a into main 2026-05-10 00:13:32 +00:00
core-lead deleted branch docs/fix-minimap-audit 2026-05-10 00:13:32 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
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: molecule-ai/molecule-core#201
No description provided.