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

Change Button Color Contrast to Meet Accessibility Requirement #37

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

shangguanwang
Copy link
Collaborator

Description

All buttons currently have a light blue background which doesn't pass Lighthouse's accessibility requirement.

Related Issue

Accessibility

Acceptance Criteria

  • Modify button background color to a darker black to meet Lighthouse's accessibility requirement.

Type of Changes

Type
πŸ› Bug fix
✨ New feature
βœ“ πŸ”¨ Refactoring
πŸ’― Add tests
πŸ”— Update dependencies
πŸ“œ Docs

Updates

Before

lighthouse-accessibility-test-before

After

lighthouse-accessibility-test-after

Testing Steps / QA Criteria

To pull our branch locally, run git pull origin btn-color-contrast
Next, run git checkout btn-color-contrast
Launch the app with npm start
Install Lighthouse Chrome extension if you don't have it already
Go to the Add Item page, or any page with the button. Open the console on the browser, go to Lighthouse tab and Generate a Lighthouse report on the "Accessibility" category, expect to see a rating of 100.

@github-actions
Copy link

github-actions bot commented Oct 8, 2023

Visit the preview URL for this PR (updated for commit 18d6cbf):

https://tcl-63-smart-shopping-list--pr37-btn-color-contrast-9w9f97ef.web.app

(expires Sun, 15 Oct 2023 21:04:47 GMT)

πŸ”₯ via Firebase Hosting GitHub Action 🌎

Sign: 33cd697c9ae2cd133e732431f4d096ea24d56582

Copy link
Collaborator

@rdpfeifle rdpfeifle left a comment

Choose a reason for hiding this comment

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

Looks good!

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.

None yet

4 participants