Skip to content
OverviewSolutions
1
Comments
4
Cristian Melo
@cristianmeelo

All comments

  • Daniel 🛸•44,310
    @danielmrz-devSubmitted over 1 year ago

    That's my first completed challenge. I know it's not completely correct, but it's okay. I'm gonna keep learning and improving :)

    QR Code Challenge - Test

    1
    Cristian Melo•50
    @cristianmeeloPosted about 1 year ago

    Fala Daniel ! Muito muito bom teu design. Parabéns.

    Tu poderia me contar um pouco a respeito da html semântico, por que tu optou por um h2 e não por um h4 ?

    • h2 vejo como um subtítulo da section/page, mas pode fazer sentido porque temos apenas um header grandão na página.

    • porém, por tamanho do conteúdo disponibilizado, font-size:15px no style-guide, h4 pode fazer sentido.

    1
  • Cristian Melo•50
    @cristianmeeloPosted about 1 year ago

    Fala Décio ! Muito muito bom teu design. Parabéns.

    Tu poderia me contar um pouco a respeito da html semântico, por que tu optou por um h1 e não por um h4 ?

    • h1 vejo como título da section/page, mas pode fazer sentido porque temos apenas um header na página.

    • porém, por tamanho do conteúdo disponibilizado, font-size:15px no style-guide, h4 pode fazer sentido.

    0
  • Cristian Melo•50
    @cristianmeeloPosted about 1 year ago

    Hello Samarth! You did a great job! Congratulations on finishing the challenge. I would also like to supplement your reading about semantic html, take a look at when to use tags like h1-h6, p, main. This will make your code more readable.

    Feel free to see/comment in my solution too.

    Marked as helpful
    1
  • Cristian Melo•50
    @cristianmeeloPosted about 1 year ago

    Oi Pamella, vi que tu terminou o desafio. Ficou muito bom! Podia me contar uma pouco da tua escolha em usar um h2 ao invés de um h4 ou h6?

    Senti falta da tag main dentro body, talvez semanticamente,

    <div class="container"> essa div poderia ser main ?
    Marked as helpful
    0