@DanijelAdrinek
Posted
to start off, your body and html tags should be 100% of the height of the screen, you need to use one of the methods of vertically and horizontally centering the card inside the body, the card itself should prolly be the main tag, or if you're going to keep it outside the card, then make it so card is a section tag, you didnt add the colors for the texts, title being dark-blue, and paragraph being grey, the size of the card doesnt match the size of the card in the design, the space between the card and the content design would be more clean if it was padding on the card instead of margin of the content inside it, if you were using figma design, you can select and element, and hold alt to see how big the space between 2 elements is, you should also see how big the design in figma is, lower the size of your screen to that size, and take a screenshot (ctrl + shift + s) then you can paste that screenshot in figma, add 50% transparency to it, and put it on top of the design to see if it matches or not, thats how you will do the job once you are a frontend web developer.
You should also get familiar with css variables, on small projects like these, they dont make that big of a difference, but on big projects, they are a life saver!
anyway, it is still a good project, and I hope you found this helpfu, in case you want to check my solution out and learn from it, here is the link: https://www.frontendmentor.io/solutions/just-css-and-html-its-too-small-of-a-project-to-need-ro-add-anything-Lo9rFod9_5
if you have any questions or need any additional help, feel free to reach out, I'm here to help :)
Hope you found this comment helpful, and happy coding :D
Marked as helpful