Dave
@Guitar8634All comments
- @Mar1362Submitted 2 months ago@Guitar8634Posted 2 months ago
Nice use of semantic HTML and pseudo-classes in CSS. Everything is clear!
There is a slight indenting of the
<h1>
and<p>
elements in the hover state which you might try to include as well.Marked as helpful0 - @lola-iloriSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I’m most proud of launching my first-ever live website. It’s exciting to step away from learning and test my abilities in real-time.
I’m still learning, so I know I’ll make mistakes, but I’m eager to learn from each one.
What challenges did you encounter, and how did you overcome them?CSS was my greatest challenge. There are a lot of best practices, do's and don'ts that I didn’t fully adhere to. Additionally, I found myself re-inventing the wheel a lot while styling certain elements. Thankfully, research and AI came to the rescue.
What specific areas of your project would you like help with?CSS, definitely! There are a lot of nuances and styling techniques that are still new to me. I'm eager to learn them all and would appreciate any guidance in these areas.
@Guitar8634Posted 3 months agoNice job matching the solution to the design. The code is clearly readable. Perhaps consider using an
<h1>
instead of<h3>
for semantics? Does having an<h3>
imply that there is an<h2>
and<h1>
before it? I could be wrong about this.As a next step, consider making use of variables in the CSS for the colors listed.
well done!
0