PLooks really good but you are missing the " image in the Daniel card
0
Design Comparison
SolutionDesign
Report
0Accessibility
issues0HTML
issues
Moises's questions for the community
Practicing CSS Grid, I really like this property and I put my knowledge into practice, any contribution and help is welcome. Thanks.
Community Feedback
P
Mohit Menghnani55 | Posted 6 days agoThis looks really great. I have a minor things that I was thinking upon
1) All img tags seem to have the photo class. Can you just use the img tag for the style?
2) For the alt text, should it just say Daniel Clifford or something like "image of Daniel Clifford". I am curious on this one myself
But I love this work, congrats and all the best. Cheers!
0Hello @mmenghnani
1.- Yes, it would work perfectly adding the styles to the img tag, in fact I removed the class. 👍
2.- Of course, for accessibility issues if it would be better to complement the alt attribute with more information, for now I will leave it with Photo of Daniel Clifford ✌
Thanks for the input.
0
Give some feedback to devmoises about their solution...
Join our Slack community
Join over 50,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!