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

Alt text #1272

Merged
merged 3 commits into from
Dec 7, 2018
Merged

Alt text #1272

merged 3 commits into from
Dec 7, 2018

Conversation

Link2Twenty
Copy link
Contributor

@Link2Twenty Link2Twenty commented Dec 6, 2018

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

Added alt tags to the home page for a11y compliance.

Related Tickets & Documents

Resolves #1212

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

`alt` tag is blank at @Zhao-Andy's request
`alt` tag is blank at @Zhao-Andy's request
Added `alt` tag for a11y compliance
@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Dec 6, 2018
Copy link
Contributor

@maestromac maestromac left a comment

Choose a reason for hiding this comment

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

Thanks for this PR! LGTM!

@pr-triage pr-triage bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels Dec 7, 2018
@maestromac maestromac merged commit d387f1a into forem:master Dec 7, 2018
@pr-triage pr-triage bot added PR: merged bot applied label for PR's that are merged and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels Dec 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants