@denisaorlikova91
Posted
Even though placing your styles within a `` tag for the main content, however the <article>
tag is not semantically correct. A better solution would be to use <div class="card">
and using .card
selector in your CSS for styling. This makes both your HTML and CSS easier to understand, maintain and reuse.
Although at first glance your solution might seem visually correct, at the closer look your paddings within the card are off. It's important to inspect the design in detail and calculate correct dimensions for all white space. Example: the card width is 320px, the QR img width is 288px. (320-288)/2=16 So the correct padding for your card is 16px. From here you can work out the white space between the rest of the elements within the card which is usually in multiples of the same number, in this case 16px. Don't forget to double check your font-sizes too. In this case the correct h1 size would be 22px.
Overall a good job for a first project, though! It takes time to train your eye for all the details and to take up the habits that promote well readable and reausable code. Hope this feedback helps & good luck!
Marked as helpful