Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

[#308] Some Images missing "alt" property #319

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[#308] Some Images missing "alt" property #319

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1ddd00a
[#308] Some Images missing "alt" property
abdqudus Dec 17, 2023
a7aa244
[#308] Some Images missing "alt" property
abdqudus Dec 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/components/Testimonials.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ export function Testimonials() {
testimonial.author.username +
'.png'
}
alt=""
alt={`${testimonial.author.name} image}
Copy link
Member

@kkrishguptaa kkrishguptaa Dec 17, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

Suggested change
alt={`${testimonial.author.name} image}
alt={`${testimonial.author.name}'s profile picture`}

Copy link
Author

@abdqudus abdqudus Dec 17, 2023

Choose a reason for hiding this comment

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

OK, I will do that now

width={56}
height={56}
/>
Expand Down