Skip to content
Submitted over 2 years ago

Responsive landing page using tailwinds css, node express server

Ephraim Malinga•20
@malep2007
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Intro section with dropdown navigation
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

I'd like more guidance on

  • the dropdown implementation,
  • also looking to have some feedback on the responsive layout, I'm not sure I got it right.
  • and any other areas I can improve are more than welcome

Community feedback

  • Andrew•380
    @andreich1980Posted over 2 years ago

    Hey Ephraim

    Since you're using Tailwind CSS you might want to take a look at the Headless UI library. It has drop-down and many other cool component. But it is only for React or Vue. But would save you so many time!

    Otherwise you could use Alpine JS (it's a lightweight js framework). It's extremely useful for cases like yours.

    Other than that seems you haven't implemented any active states (on hover), focus states for links and buttons also look off of the design.

    You shouldn't mix hidden and visible classes - these are different: hidden sets display to none and visible sets visibility to visible. If you want to hide something on mobile then show on wider screens - user hidden md:block or ... md:flex or ... md:grid whatever display type you need.

    In the #hero section you mix up grid and flex, I think you'd better use one thing at a time.

    Marked as helpful
    1
    Ephraim Malinga•20
    @malep2007Posted over 2 years ago

    @andreich1980 thanks for the feedback. Let me work on it and improve my submission.

    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