Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

QR Code component HTML/SCSS

@cauesooouza

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


as easy as pie

Community feedback

Adriano 34,090

@AdrianoEscarabote

Posted

Oi Cauê Souza, tudo bem?

Gostei muito do resultado do seu projeto, mas tenho algumas dicas que acho que você vai gostar:

Como você sabe, este projeto é baseado apenas em um componente de página, portanto, não é necessário h1! pois não sabemos se esta página terá um componente mais importante, mas é sempre bom prevenir erros de acessibilidade então acho que seria bom você adicionar um h1 nesse componente, além de ser uma boa prática para quando você está desenvolvendo sites maiores, não se preocupe, esqueça o h1.

main {
    min-height: 100vh;
}

Use min-height em vez de height, pois quando o conteúdo for além de main o conteúdo acabará sendo ocultado! faça o teste aumentando a altura do article!

o resto é ótimo!

espero ter ajudado... 👍

0

@cauesooouza

Posted

@AdrianoEscarabote fica tranquilo que questão de acessibilidade e seo, esta bem otimizado, e enquanto ai height, não faz diferença pois se fosse uma pagina com outros elementos, nao seria feito dessa forma utilizando 100vh, ele seria em tamanho relativo, ocupando somente o espaço necessario, o 100vh foi utilizado somente para questões de centralização, obrigado pela sua ajuda mas não é necessario alterações

1
Adriano 34,090

@AdrianoEscarabote

Posted

@cauesooouza boaa

0

@AltairCGS

Posted

To properly center your content to your page, you will want to add the following to your Body element (this method uses CSS Grid):

body {
    min-height: 100vh;
    display: grid;
    place-content: center;
}

A little advice that they gave me before :3 I hope this helps! Keep up the good work!

0

@cauesooouza

Posted

@AltairCGS ins't a good a idea give height to body... when you need to center a element in axis x use: margin: 0 auto; when you need to center a element in axis y use: flexbox

0

Please log in to post a comment

Log in with GitHub
Discord logo

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