Skip to content

PRXT - feat: improve TooltipIconButton accessibility#1426

Closed
punk6529 wants to merge 1 commit intomainfrom
codex/refactor-tooltipiconbutton-component
Closed

PRXT - feat: improve TooltipIconButton accessibility#1426
punk6529 wants to merge 1 commit intomainfrom
codex/refactor-tooltipiconbutton-component

Conversation

@punk6529
Copy link
Copy Markdown
Contributor

Summary

  • convert TooltipIconButton into a semantic button that manages tooltip visibility for hover and focus states while keeping its tooltip id associated for assistive tech
  • ensure the component preserves consumer props such as aria-describedby and tabIndex when forwarding to the button element
  • add colocated tests that validate hover, focus/keyboard usage, tab order, and click behavior

Testing

  • npm run lint
  • npm run type-check (fails due to existing repository issues)
  • BASE_ENDPOINT=http://localhost npx jest components/common/TooltipIconButton.test.tsx --coverage --watchAll=false

https://chatgpt.com/codex/tasks/task_e_68c93fbbf524832186b72851c1b70a80

Signed-off-by: OpenAI <openai@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@prxt6529 prxt6529 changed the title feat: improve TooltipIconButton accessibility PRXT - feat: improve TooltipIconButton accessibility Sep 30, 2025
@prxt6529
Copy link
Copy Markdown
Collaborator

Will be done in #1497

@prxt6529 prxt6529 closed this Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants