@correlucas
Posted
👾Hello my friend Johnny, congratulations for your new solution!
Great great solution brother! I did not start yet these kind of challenges and you're finishing a lots of junior challenges, its really nice how far you've arrived. Congrats Johnny!
I've some tips for you:
If you'll really not use the attribution its better you delete it than keep it as comment, because this text makes the page takes more time to load, so its a best practice delete a comment if this is not explaining the code. To organize the icons you've used div but in this case I think is better to use <ul> <li> as a list like we usually do with social media or depending you can use also <nav>.
👋 I hope this helps you and happy coding!
Marked as helpful
@johnnysedh3lllo
Posted
hmmmm i guess I'll take note of that, didn't know it affected the page load time. and also in terrms of using the <ul> <li>, i'll try it out in my next practice project. thanks again brother, your feedback is as helpful as always. thank you
