Skip to content

Change alt text for Marvista community council image#3435

Merged
Wny-Duong merged 1 commit intohackforla:gh-pagesfrom
ericvennemeyer:change-alt-text-marvista-3107
Jul 29, 2022
Merged

Change alt text for Marvista community council image#3435
Wny-Duong merged 1 commit intohackforla:gh-pagesfrom
ericvennemeyer:change-alt-text-marvista-3107

Conversation

@ericvennemeyer
Copy link
Member

Fixes #3107

What changes did you make and why did you make them ?

  • Changed alt text for Marvista community council image on citizen engagement page from "Join Us! marvista.org | mar vista community college logo" to "Marvista Community Council"
  • This change was made to better conform to Web Content Accessibility Guidelines

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

No visible changes to website.

@github-actions
Copy link

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b ericvennemeyer-change-alt-text-marvista-3107 gh-pages
git pull https://github.com/ericvennemeyer/website.git change-alt-text-marvista-3107

@github-actions github-actions bot added Feature: Accessibility Issues that would broaden website accessibility P-Feature: Citizen Engagement https://www.hackforla.org/citizen-engagement role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less Size: Good second issue labels Jul 27, 2022
@arpitapandya arpitapandya self-requested a review July 28, 2022 01:08
@arpitapandya
Copy link
Member

Availability: 6 Hours
ETA: EOD 2022-07-28

Copy link
Member

@arpitapandya arpitapandya left a comment

Choose a reason for hiding this comment

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

The changes looks good! Great job!! @ericvennemeyer

@Wny-Duong Wny-Duong self-requested a review July 28, 2022 23:36
@Wny-Duong
Copy link
Member

Availability: 1 Hour
ETA: 7/28

Copy link
Member

@Wny-Duong Wny-Duong left a comment

Choose a reason for hiding this comment

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

Changes are good! Keep up the good work.

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

Labels

Feature: Accessibility Issues that would broaden website accessibility good first issue Good for newcomers P-Feature: Citizen Engagement https://www.hackforla.org/citizen-engagement role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change alt text for Marvista community council image in citizen engagement html file

4 participants