Skip to content
Submitted over 1 year ago

Responsive page, CSS flexbox

Nicolas Leon•70
@Leonngc
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Space tourism multi-page website
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

O projeto foi de um nível intermediário para mim, porém a principal dificuldade foi no menu mobile, onde ainda considero que posso melhorar. Os erros que aqui forem apontados corrigirei nesse e nos futuros projetos.

Community feedback

  • Saad Hisham•1,750
    @Saad-HishamPosted over 1 year ago

    Hey there! I wanted to start off by saying great job on tackling this challenge. I do have a few tips that might help improve it even more. First, I would suggest removing the background-repeat and sizing it to cover the entire page. Additionally, attaching it as fixed will prevent it from fading as you scroll. Keep up the good work!

    body {
    background-repeat: no-repeat;
    background-size: cover;
    background-attachment: fixed;
}

    hope that was useful happy coding

    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