-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Header formatting on email version of TWiR is not line breaking correctly #3592
Comments
I think the alt-text on the icon is actually hurting here (since we repeat that exact text to its right), and suppressing it would give a better outcome. |
@mariannegoldin / @ericseppanen do either of y'all have interest in trying to fix this? We don't wanna suppress the alt-text, but that's definitely a less than optimal experience for users. If you don't have time or expertise, I'll add that |
I think my preferred fix is still to lose the alt-text. Can you explain why you disagree? I don't think it adds any value, because that exact text is already there. If the image doesn't load, nothing is lost. |
I just went back and looked at this again and realized that I'd completely missed that we include it on 119 in addition. I'd really like to hear from someone with a deeper accessibility understanding. Even though the text is repeated just next to it, removing it would cause a problem for anyone relying on that alt text. |
I think it's reasonable to set
pure decoration is defined as:
Specifically setting the |
If we're being very pedantic, the alt text is currently wrong anyway. The HTML Standard provides some guidance on what the alt-text should be:
And here, it already changes the meaning of the page. The image being replaced is not "This Week in Rust", it's "The This Week in Rust Logo". However, in this case, setting
So yeah, +1 for setting the alt text to an empty string. If you want to be explicit about telling screenreaders to skip over the image entirely, you can additionally set the |
Thank you @bennyvasquez for the fix! |
The title image and header of the email are not breaking correctly in the generated HTML code for the TwiR email issue.
The break should be placed after the "This Week in Rust" title image. Instead, the line is currently breaking after the "This" in the text header.
The email code is being generated by the
optimize-email
Makefile job, which uses the website code (via themake build && make generate-website && make host-content
command) to create an email usingjuice
.Narrow window:
Wide window:
The text was updated successfully, but these errors were encountered: