Submitted almost 2 years ago
Notification page with vanilla js
@Clement1kalu-okereke
Design comparison
SolutionDesign
Solution retrospective
Hey 👋 Community I just finished this challenge and I learned a handful of new concepts.
Building it was fun and I would love any form of Constructive criticism
Thanks
Community feedback
- @AdrianoEscarabotePosted almost 2 years ago
Hi huxixi, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
I noticed that the content was growing a lot at higher resolutions, and this was hurting the design of the project to solve this we can do this:
.notification__main { max-width: 900px; }
<div class="time">5m ago</div>
avoid using too many divs, in this case, prefer to use a
span
tagThe rest is great!
I hope it helps... 👍
1@Clement1kalu-okerekePosted almost 2 years ago@AdrianoEscarabote Thanks for the update
1
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