@AbNassif
Posted
Great work!
Looking to hire developers?
The only thing I find weird is the box shadow settings that were in sketch file but hopefully everything is like it should be
@AbNassif
Posted
Great work!
@karenefereyan
Posted
I love it
Hello, Patrick! 👋
Firstly, congratulations on becoming mentor of the week! Your solution looks very good and also responds well. Here's my few tips:
alt
attributes for the .users
avatars, as they don't add any extra context for screen reader users. Since your images are decorative your alt
text should be provided empty (alt="") so that they can be ignored by assistive technologies. Informations provided by .name
and .graduate
perfectly describe it itself.Good luck with that, have fun coding! 💪
@tediko Thanks, if I helped anyone that last week that is all that matters. I know I worked on these projects multiple times 9+ months before I started uploading any of them and still end up re doing them multiple times
I will keep in mind the screen reader issues and I know I need to study the whole accessibility thing with the changes to EU laws
I did mix classes with my IDs but I have been giving things IDs even if I do not style with them to keep the code my head better. I know I am creating global Dom objects but with practice projects I am not worried about having IDs on most elements at this point. But every challenge I seem to do a little different so we will see what I end up with long term
@Pascal488
Posted
Hey bro congrats i've checked all of your projects and end up appreciating your effort big up bro,i wish we could be friends here is my Whatsapp number ping me if want +255789310923
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord