Skip to content

1337 defect pie chart is not accessible#1443

Open
wlh3903 wants to merge 4 commits intodevfrom
1337-defect-pie-chart-is-not-accessible
Open

1337 defect pie chart is not accessible#1443
wlh3903 wants to merge 4 commits intodevfrom
1337-defect-pie-chart-is-not-accessible

Conversation

@wlh3903
Copy link
Copy Markdown

@wlh3903 wlh3903 commented Apr 1, 2026

Added more descriptive aria-label to Pie Charts

  • No discrepancies across browsers (ex: chrome vs safari)
  • Accessibility functions
  • Pages can scale without distorting page
  • No dead links
  • Navbar is consistent across the site
  • Pages are screen-reader accessible
  • Contrast meets standards for accessibility
  • Pages are keyboard accessible
  • Code is cleaned up and bug-free (ex: debug statements removed)

@wlh3903 wlh3903 requested review from JackyFTW and ejt7508 April 1, 2026 15:45
@wlh3903 wlh3903 self-assigned this Apr 1, 2026
@wlh3903 wlh3903 linked an issue Apr 1, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Member

@JackyFTW JackyFTW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just make that small change!

Copy link
Copy Markdown
Member

@JackyFTW JackyFTW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@JackyFTW JackyFTW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Defect: Pie Chart is Not Accessible

2 participants