Skip to content

refactor(web): remove generated result React.FC - #40281

Closed
lyzno1 wants to merge 1 commit into
codex/refactor-generated-result-css-iconfrom
codex/refactor-generated-result-component
Closed

refactor(web): remove generated result React.FC#40281
lyzno1 wants to merge 1 commit into
codex/refactor-generated-result-css-iconfrom
codex/refactor-generated-result-component

Conversation

@lyzno1

@lyzno1 lyzno1 commented Aug 9, 2026

Copy link
Copy Markdown
Member

Summary

  • Replace React.FC with an explicit Props annotation on the generated result component.
  • Replace the React namespace import with the named memo import.
  • Preserve the memoized export and every render path.

Dependency

This PR is stacked on #40279 only because all three PRs intentionally touch the same component in review order. It has no accessibility or icon behavior dependency.

Behavior contract

The component accepts the same props and remains memoized. There are no state, service, event-handler, or test changes.

Visual regression

No visual change is expected: JSX, DOM, classes, styles, layout, icons, ARIA attributes, and handlers are unchanged relative to #40279.

Reviewer focus: confirm the diff is limited to the React import, explicit props annotation, and memo export.

Validation

From Codex

@github-actions github-actions Bot added the web This relates to changes on the web. label Aug 9, 2026
@lyzno1

lyzno1 commented Aug 9, 2026

Copy link
Copy Markdown
Member Author

Closing during the accessibility stack audit. This layer only changes component typing or icon implementation, adds no user-visible accessibility contract, and carries independent visual/review risk. It can be reconsidered later as a separate cleanup, not as an a11y dependency.

@lyzno1 lyzno1 closed this Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant