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
x/website: missing alt tags on gopher images in 3 pages #39861
Comments
/cc @dmitshur |
Thanks. Since you already know specific pages that have problems, it would help a lot if you could mention them. |
From that description, I understand it's these URLs:
They are indeed missing alt attribute on the gopher images. Please let us know if I misunderstood, @nel417. /cc @andybons |
@dmitshur yes, looks perfect. Those are the images and URLs I was seeing that were missing a few alt tags. |
Made a change here : https://go-review.googlesource.com/c/website/+/240257 Not sure where the help file is located at. Any guidance would be appreciated. |
Change https://golang.org/cl/240257 mentions this issue: |
That file is still in the main Go repository, here. |
Change https://golang.org/cl/240258 mentions this issue: |
Hi @dmitshur - thanks for the quick review. I made changes to the commit message and I think I got the tests passing now after running |
Thank you. I can re-run the trybots now. |
Hi @dmitshur - Would you want the alt tag for the footer updated to also reflect a lowercase 'g'. i.e., change the alt tag from "The Go Gopher" to "The Go gopher" in the alt attribute for the footer on each page? |
@nohe427 I didn't see there was already precedent for it on the same page, thanks for spotting that. Looks like it's not consistent with the blog post. Oh well, we're going with an empty string alt attribute, so let's not change other things for now. |
Hi @dmitshur - thanks for the quick turn around! Updated the specified pages with empty alt tags. Let me know if there is anything else needed before submitting. |
Thanks @dmitshur for the +2 to review. It seems I am unable to submit with the codereview. Is there something I am missing? |
No worries, I can submit for you. (You can read more about Gerrit access here.) I just wanted to give you a chance to update the commit message if you wanted. I'll submit soon. The update will be live on golang.org after the next time it's redeployed. The change in the /help/ file will take longer, because it's in the main Go repo, so it'll get updated when 1.15 is out. |
This CL adds alt tags for the gopher images in docs, for better accessibility. The HTML spec indicates that purely decorative images have empty alt tags, hence, the empty alt tags here. Fixes golang/go#39861 Change-Id: I5761e5c6c0005a6974e96a21263605fbfe8f27fc Reviewed-on: https://go-review.googlesource.com/c/website/+/240257 Reviewed-by: Alexander Nohe <alex.nohe427@gmail.com> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
@dmitshur - ahh okay. Many thanks! Thank you for the quick reviews and turnaround time. |
Thank you for contributing a fix for this issue! |
This CL adds alt tags for the gopher images in docs, for better accessibility. The HTML spec indicates that purely decorative images have empty alt tags, hence, the empty alt tags here. Fixes golang/go#39861 Change-Id: I5761e5c6c0005a6974e96a21263605fbfe8f27fc Reviewed-on: https://go-review.googlesource.com/c/website/+/240257 Reviewed-by: Alexander Nohe <alex.nohe427@gmail.com> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
No, This is a website issue
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
A few pages on the Go website, Documents, Project, And Help, are scoring a 90 SEO and 93 accessibility because of missing alt tags. Via Lighthouse
What did you expect to see?
Images with alt tags
What did you see instead?
No alt tags, most pages have 100 accessibility and SEO, but these 3 pages were off.
The text was updated successfully, but these errors were encountered: