@DavidMorgade
Posted
Hello Donn, congrats on finishing the challenge!
As Aurel said, using paddings and margins is normal in any kind of projects, try to follow the styling guide to get your project to look more like the FEM solution, remember that you don't have to have it 100% the same.
I would like to give you a little recommendation, you can use flex-direction: column
on your id=main
for mobile devices, for example having it on a media querie:
#main {
@media(max-width: 768px) {
flex-direction: column;
}
}
With this code your component will look like the solution for mobile devices!
Another advice is to take advantage of the <picture></picture>
tag, in this project you are provided with two images, one for desktop and other for mobile, you can use de picture tag along the <source>
tag to put both images and use them conditionaly depending on the width of the screen.
Hope my feedback helps you for future challenges, good job!