Skip to content
Submitted almost 2 years ago

Notification page with vanilla js

Okereke Clement Kalu•170
@Clement1kalu-okereke
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Notifications page
  • HTML
  • CSS
  • JS
2junior

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

  • Adriano•38,230
    @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 tag

    The rest is great!

    I hope it helps... 👍

    1
    Okereke Clement Kalu•170
    @Clement1kalu-okerekePosted almost 2 years ago

    @AdrianoEscarabote Thanks for the update

    1

Please log in to post a comment

Log in with GitHub

Join 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