Skip to content
Submitted about 2 years ago

Responsive webpage using grid and flexbox

CalfMoon•100
@CalfMoon
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...News homepage
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

The button(label) responsible for opening menu in mobile version has a problem where its background changes on click. This was supposed to be a feature for tabbing but ended up being a problem.

Community feedback

  • Tryt4n•960
    @Tryt4nPosted about 2 years ago

    It's becasue you write it to be color changed: .nav-toggle-label:hover { background-color: rgba(0, 0, 0, 0.1); }

    Also there is no such a thing like label type="button" You should use button instead of label. label element is for inputs to describes what that input is doing. Something like header for input, and that's why label has for="" attribute. This is where you type id of input that should be labelled by that label.

    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