Mobile First Responsive Data Storage Component Built With CSS Flexbox.
Design comparison
Solution retrospective
here's another one from me π. this one really had me dusting my knowledge on CSS positiong. also learnt how to make speech bubbles, so that was cool.
what do you guys think of my code tho? would love to hear your thoughts on that.
Community feedback
- @correlucasPosted about 2 years ago
πΎ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 useddiv
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 helpful1@johnnysedh3llloPosted about 2 years agohmmmm 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 you0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord