Flexbox

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey @edvanbarbosa, some suggestions to improve you code:

  • Implement a Mobile First approach 📱 > 🖥

  With mobile devices being the predominant way that people view websites/content. It is more crucial than ever to ensure that your website/content looks presentable on all mobile devices. To achieve this, you start building your website/content for smaller screen first and then adjust your content for larger screens.

  • The car images/icons serve no other purpose than to be decorative; They add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.

  • The headings are being use incorrectly. For this challenge, each heading is equally as important. So best option, is to use <h2> Heading, because it will give each card the same level of importance and it's reusable.

  Happy Coding! 👻🎃

  Marked as helpful

  0

  ---

  

  Edvan Barbosa•320

  @edvanbarbosaPosted about 2 years ago

  @vcarames

  Sobre a hierarquia de títulos, uso assim porque me foi ensinado desse jeito, porém depois da sua observação pesquisei e vi que de fato você está correto. De acordo com a w3schools, o <h1> só pode ser utilizado uma vez por página.

  Sobre o Mobile First, acabei esquecendo desse ajuste, pois como falei na descrição, essa é a minha primeira vez criando um projeto sem orientação.

  Agradeço a atenção e irei retificar os erros. Muito obrigado pelas dicas ; )

  0

  

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  @edvanbarbosa

  (Google Tradutor usado)

  Só pode haver um <h1> Cabeçalho> por página. Assim, depois de atribuí-lo a algo, você não pode mais usá-lo na mesma página novamente.

  Para este desafio, o <h2> Heading é a melhor escolha, pois dá a cada título o mesmo nível de prioridade e é reutilizável.

  Aqui estão alguns links para ajudá-lo a entender melhor como usar títulos.

  https://udn.realityripple.com/docs/Web/HTML/Element/Heading_Elements

  https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements

  https://www.codecademy.com/resources/docs/html/headings

  0

  

  Edvan Barbosa•320

  @edvanbarbosaPosted about 2 years ago

  @vcarames ,

  Dei uma atualizada no comentário ; )

  0