@Lb2468
Submitted
@dawidPoznanski
@Lb2468
Submitted
@dawidPoznanski
Posted
Hi, great job!
But I would like to add some small changes to your code.
Let's start by changing <h2>HTML & CSS foundations</h2>
It's not necessary to use the h2
element you can do it by <a href="#">HTML & CSS foundations</a>
because its link to a course. And you can delete all lines from 21 to 31 in your HTML file.
About CSS file you should use a Prettier format or something similar to add space between classes like you add class
body {
some CSS code ...
}
main {
some CSS code ...
}
And try to not add empty spaces like in lines 10 or 17-18. Anyway, congrats to the end of this challenge :D
Marked as helpful
@mark-escosura
Submitted
@dawidPoznanski
Posted
Nice work 😁
If u want to get this qr card component to the center of the web page u should use display:flex flex-direction:column; align-items:center;
or try to make another div for your card component and add this style to your new div. In <img>
you should put alt=""
with some description
Marked as helpful