c2pa-js
c2pa-js copied to clipboard
fix(#145): ManifestSummary/Tooltip/Popover accessibility
Changes in this pull request
Tooltip component used in ManifestSummary should:
- Trigger should be keyboard accessible as a button.
- Trigger should be labeled by the Tooltip and the header.
- Popover for Tooltip should expand on focus and close on blur.
- Popover should hide with Escape key and toggle on Trigger click.
- Fix text color contrast for --cai-secondary-color.
- Fix color contrast for --cai-icon-fill
- Add focus outline color and outline-offset to Tooltip button and View More
- Use
<section>
with aria-labelledby for PanelSection. - Add role="heading" and a heading level for the .heading-text element within PanelSection
- Add role="img" and appropriate aria-label for each of the Icons
Types of changes
- [x] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist
- [ ] All applicable changes have been documented
- [ ] Any
TO DO
items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment