Vincenzo Marcovecchio• 1,085
@VincenzoMarcovecchio
Posted
Hey Matt! Thanks! as always great suggestions. Ill'keep that in mind, i have to make sure everything is in place before submitting a solution :)
1
Looking to hire developers?
Submitted
@VincenzoMarcovecchio
Help improve my work, any feed back appreciated
@VincenzoMarcovecchio
Posted
Hey Matt! Thanks! as always great suggestions. Ill'keep that in mind, i have to make sure everything is in place before submitting a solution :)
@mattstuddert
Posted
Your solution looks great, Vincenzo! I've only got a couple of small recommendations:
footer
would be clickable elements if this were a real site, so I'd recommend wrapping them in anchor tags.footer
is uppercased in the HTML. Be aware that screen reader software will read this out letter-by-letter instead of reading out the word. This makes that heading inaccessible to screen reader users. Instead, I'd write it normally in your HTML and then use text-transform: uppercase;
in your CSS to visually uppercase it to match the design.I hope that helps. Keep up the great work! 👍
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