@DanijelAdrinek
Posted
the size of your card is not the same as the size of the design, you didnt add any box-shadow, the image is too big, the font-size of the text is too big, and also in px instead of rem, we usually use rem because that allows users to choose how big the text on our page will be trough their browser settings.
usually in frontend development, you will have to see the size of the design, lower the size of your screen to that size, and take a screenshot (ctrl + shift + s), and compare it to the design in figma, and make sure it matches the design in every way, I believe here you have the perfect training ground for a task like that, and you get to show off your work to potential employers, so I suggest you start learning to do that, in css you didnt use any css variables, and sure, they arnt too useful on projects this small, but in bigger projects, they are a real life-saver.
div with a class of box should be a tag of main to help the screen readers, the container shouldnt be the size of height:100vh; you dont even need the container, just add height: 100%; to body and html tags, and center the card in the body tag.
anyway, still a good little project, I enjoyed going trough all the little things to try to help, if you want, here is how I finished the project, I hope you get to use it to learn from it and become a better web developer: https://www.frontendmentor.io/solutions/just-css-and-html-its-too-small-of-a-project-to-need-ro-add-anything-Lo9rFod9_5
if you got any questions you wish to ask, feel free to reach out, I'm always here to help :)
Hope you found this comment helpful, and happy coding friend :D
