Skip to content
Submitted 4 months ago

notifications page main

Iyalekhue osagie•190
@Thirty-2
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

What are you most proud of, and what would you do differently next time?

The use of tailwind css

What challenges did you encounter, and how did you overcome them?

tailwind css installation

What specific areas of your project would you like help with?

nothing for now

Community feedback

  • Krushna Sinnarkar•1,100
    @krushnasinnarkarPosted 4 months ago

    Hi @Thirty-2,

    Congratulations on successfully completing the challenge!

    Your solution looks nice, though there are a couple of things you can improve, which I hope will be helpful:

    1. Width Classes: You're using too many width classes for the #Container. It's still occupying too much width on desktop screens. Simplify it by just using max-w-2xl Tailwind class instead of all these classes: w-fit sm:w-fit md:w-fit lg:w-fit xl:w-fit. This will ensure the container has a maximum width and remains responsive.

    2. Background Color: Add a background color to the body to enhance the visual appeal and match the design specifications.

    I hope you find this helpful.

    Feel free to reach out if you have more questions or need further assistance.

    Happy coding!

    Marked as helpful
    0

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