@gregoriofrancisco99
Posted
Hello, Reza! Gotta tell you, loved your solution. And the comparison screenshot speaks by itself! Congratulations! Even tough the design looks really good, I took a look at your code and I will raise some points.
You should try to wrap all your main content inside a <main>
.
And if you have more than one <section>
you should consider wrapping them inside an <article>
.
<h1>
are the top level heading in a page, and there should be only one of it. and <h2>
is what follows it on the heading hierarchy. Clique here to get to know them all (from <h1>
to <h6>
) and what are they meant for.
And I saw you used a <button>
for the sign up. And it's not really wrong, but for me, I think you should use an <a>
. Because it seems it will take the user somewhere else, or open some other content.
I saw you used an <ul>
for the last card. That was awesome!!! 🎉
Well, those are my comments for this solution. And remember... Keep coding 💻
Marked as helpful
@rezajaber
Posted
@gregoriofrancisco99 Hey my friend! I want to thank you so much! This was not just really helpful but it also showed me that you really did look over my code and took some time to help me. This just feels awesome and I really appreciate it🔥
I will definitely remember these tips and use them for the next project! Again a huge thanks and stay active ✌️