Skip to content

docs: define interaction modality architecture - #5734

Merged
cixzhang merged 1 commit into
mainfrom
docs/interaction-modality-architecture
Aug 31, 2026
Merged

docs: define interaction modality architecture#5734
cixzhang merged 1 commit into
mainfrom
docs/interaction-modality-architecture

Conversation

@cixzhang

Copy link
Copy Markdown
Contributor

Why

Components already share modality tracking and focus primitives, but the boundary between last-input modality and component focus ownership was implicit. That made programmatic-focus behavior easy to reinterpret component by component.

What

Adds one current architecture record that:

  • defines programmatic focus visibility by the last input modality;
  • separates shared modality state from component-owned focus paint;
  • preserves existing family variation and state semantics; and
  • names change coupling and verification evidence, including PR fix(chat): show composer focus ring for keyboard #5648 as a benchmark rather than authority.

Risk

Documentation only. No runtime behavior, visual design, or public API changes.

Testing

  • pnpm exec prettier --check docs/architecture/interaction-modality.md
  • pnpm check:knowledge
  • pnpm check:changesets
  • pre-commit check:repo

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
astryx Ready Ready Preview Aug 31, 2026 2:38am

Request Review

@github-actions github-actions Bot added the needs:spec-owner-review Current knowledge records await owner approval label Aug 31, 2026
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Aug 31, 2026
@cixzhang
cixzhang marked this pull request as draft August 31, 2026 02:23
@cixzhang
cixzhang force-pushed the docs/interaction-modality-architecture branch from f0d0272 to 72f0f8b Compare August 31, 2026 02:32
@cixzhang
cixzhang force-pushed the docs/interaction-modality-architecture branch from 72f0f8b to 716204b Compare August 31, 2026 02:35
@cixzhang
cixzhang marked this pull request as ready for review August 31, 2026 02:37
@cixzhang

Copy link
Copy Markdown
Contributor Author

/approve-spec 716204b

@cixzhang
cixzhang enabled auto-merge (squash) August 31, 2026 02:38
@github-actions github-actions Bot removed the needs:spec-owner-review Current knowledge records await owner approval label Aug 31, 2026
github-actions Bot added a commit that referenced this pull request Aug 31, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Analysis Report

📚 Storybook Preview

View Storybook for this PR
GitHub Pages may take up to a minute to hydrate after deploy.

🧪 Sandbox Preview

View Sandbox for this PR
GitHub Pages may take up to a minute to hydrate after deploy.

No new or modified components detected.

Bundle Size Summary

No component packages changed.

Accessibility Audit

Status: No accessibility violations detected.


Generated by PR Enrichment workflow | Storybook | Sandbox | View full report

@cixzhang
cixzhang merged commit 457de8d into main Aug 31, 2026
29 of 30 checks passed
@github-actions
github-actions Bot deleted the docs/interaction-modality-architecture branch August 31, 2026 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant