Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addition Polish, Fix Accessibility, Remove Twitter Reference #317

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

tfreebern2
Copy link
Contributor

  • Change <h4> tags to <h1> in About section to resolve accessibility issue
  • Wrapped the <a> tag around the "Read More" in the History box so that the aria-label correctly matched the text link
  • Remove "Follow us on Twitter" in blog sidebar
  • Updated colors for "Join zCore" and "Learn More" buttons so that they pass accessibility for both light and dark backgrounds
  • Fixed text color for the Careers > Our Team title. It was white on a white background. Also fixed the margin.
  • Used this tool provided by Lighthouse to determine the correct contrast ratio in order to pass the scan https://dequeuniversity.com/rules/axe/4.8/color-contrast

image


image


image


image


image

@bastosmichael bastosmichael merged commit 6d44a9a into main Dec 13, 2023
2 checks passed
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.

2 participants