Improve something?
@tiago-jv0
Posted
There are a few things i would recommend you to learn, for exaple :
BEM methodology - is used to improve your css selectors and make your html structure more descriptive
Usage of new layout techniques like Grid or Flexbox as mentioned above
Css variables - so you can put all your repeatable css values inside a central place
Marked as helpful
Account Deleted
Hello 😊
Your current solution is a good start, however there are a couple things that could be improved:
I would recommend always making it obvious when something can be interacted with - In this case, the button. Changing the cursor to a pointer and adding a hover effect would be a great addition.
It looks like you used a table for the layout - I would recommend looking into Flexbox and CSS Grid for responsive designs - Tables should be used for data that is presented as a table, not as a layout tool. Kevin Powell has amazing courses for both of these (Flexbox Simplified, and Responsive Design Bootcamp specifically - The Bootcamp one has a section for Grid).
Your alt tag on the image should be a little bit more descriptive, generally speaking - These are meant to give an idea of what the image is, in this case it doesn't really explain what the image is.
You are missing the <main> tag after <body> - This should always be included for accessibility.
Marked as helpful
@maciekexe
Posted
@LazyDuckling Thank you I will try not to forget about it in future projects and I will try to improve this one.
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
