Skip to content

Issue #550: Fix external doc links announcement#552

Draft
lsr-explore wants to merge 1 commit into
mainfrom
lsr-explore/chore/fix-astro-check-errors
Draft

Issue #550: Fix external doc links announcement#552
lsr-explore wants to merge 1 commit into
mainfrom
lsr-explore/chore/fix-astro-check-errors

Conversation

@lsr-explore
Copy link
Copy Markdown
Contributor

@lsr-explore lsr-explore commented May 2, 2026

Fixes


Note

Changes to this PR are pending recommended resolution to the issue.


Description

  • Replace alt attribute with title attribute

Screenshots

TBD

References

@netlify
Copy link
Copy Markdown

netlify Bot commented May 2, 2026

Deploy Preview for accessiblecommunity ready!

Name Link
🔨 Latest commit 62876f8
🔍 Latest deploy log https://app.netlify.com/projects/accessiblecommunity/deploys/6a00d29ac791540008f25bab
😎 Deploy Preview https://deploy-preview-552--accessiblecommunity.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lsr-explore lsr-explore force-pushed the lsr-explore/chore/fix-astro-check-errors branch 2 times, most recently from b2dd37f to 5e4a66f Compare May 6, 2026 17:12
Copy link
Copy Markdown
Contributor

@brian-montgomery brian-montgomery left a comment

Choose a reason for hiding this comment

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

Good catch, Laurie. I didn't realize that alt wasn't useful for SVGs.

@lsr-explore lsr-explore force-pushed the lsr-explore/chore/fix-astro-check-errors branch from 5e4a66f to 62876f8 Compare May 10, 2026 18:46
@rachaelbradley
Copy link
Copy Markdown
Contributor

I agree with adding the title tag. I am not sure about how this is implemented. Typically the title is <svg ...> <title>title here</title>

The pattern used here makes sense but I haven't seen it used before with the title tag. If you have a PR, I can double check it.

@lsr-explore
Copy link
Copy Markdown
Contributor Author

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

Labels

None yet

Projects

None yet

3 participants