Hello, Sky! 👋
Congrats on finishing another challenge! I will not keep up with writing feedback if you keep posting this solutions like crazy 😅. Your solution looks very good and also responds well. I really like the slide in animations on your icons 🤩 Here's my few tips:
- Not sure if I suggested it to you but read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
- I like that your solution is responsive across different screen sizes but I'd suggest you to add
max-width: 580px;
to your.container-flex
so it doesn't stretch too much. Additionaly you have to addmargin: 0 auto;
with this to align it center.
Good luck with that, have fun coding! 💪
@Skyz03
Posted
@tediko Thanks a lot ... Will surely read more on semantic HTML. Really got your second point as once width is added we need that margin: 0 auto to align and I will update that. Thanks again ... Your feedback really motivates to do more and level up. 👍