@mattstuddert
Posted
Nice work, Fola! Definitely, a good challenge to get back into the swing of it. I hope you enjoyed looking more at JS and Git!
Your solution looks great. My only two small suggestions would be:
- Be sure not to capitalise words in your HTML. Some screen readers will read these words letter by letter, making them inaccessible. Instead, lowercase them in your HTML and use
text-transform: uppercase;
in your CSS. - There's a hover transition on the image at the moment. I would stay away from hover transitions on non-interactive elements. It's not a huge issue for this project, but something to keep in mind for the future 🙂
Keep up the great work!
@folathecoder
Posted
@mattstuddert Thank you very much Matt! I appreciate the feedbacks and I have made all the changes you suggested. I have also read up articles on why capitalization of HTML texts is not a proper way to go.
Thanks alot Matt!