greattings... i must say i love your redesign and great layout but i still found so minor mistakes nothing major :)
- your cards should be an article instead since when google index your page it will know you are making an article in your case cards
- using <section></section> for the first section please use class name like hero-section to tell other developers that this is the first thing the users will see
- needs more animations. example when hover over your cards i notices an small brigtness effect it could use a little transition fade out or in
- when you are write lists like your nav menu please use
overall great design love it alot :)
@Magic1vy
Posted
@alex931d Hello! Thank you for your feedback and kind words about my redesign. I'm glad you liked it. I would like to address your points.
1.Regarding using <article>
instead of <div>
for the cards, you're right that using <article>
could be more semantically correct. However, in this case, using <div class="card">
is an acceptable solution, as its main purpose is to group and style elements on the page, which aligns with the purpose of the <div>
tag. I will take this into consideration to improve the semantics of my code.
2.As for the first <section>
and its class, the current first section already has a semantic <section>
tag and a meaningful class name of section-interactive. Despite this, I will consider your suggestion to use the class name hero-section to make it clearer for other developers.
3.In terms of animations, it's a matter of individual developer choice, and I believe the current hover effect is consistent with the project's design.
4.Finally, about using <ul>
and <li>
for the navigation menu list, both ways of writing are acceptable and can be used depending on the developer's preference. Link about <nav>
Thank you for your time and feedback! I appreciate your help in improving my project.