CSS flex and media query ( 390px )

Community feedback

• 

  Ricky•460

  @rickyxyz

  Posted over 1 year ago

  Fellow beginner here, so take this with a grain of salt 🧂.

  I think this could be helpful for you BEM naming convention, since CSS names can get confusing quickly.

  Cheers 🍻

  21

  Marked as helpful
• 

  Melvin Aguilar 🧑🏻‍💻•61,020

  @MelvinAguilar

  Posted over 1 year ago

  Hello there 👋. Good job on completing the challenge !

  I have other suggestions about your code that might interest you.

  • Avoid using the <div> tag just for text. <div> is a generic container used for grouping content and applying styles, but it doesn't convey any semantic meaning. Similarly, avoid using <span> for entire blocks of text, <span> is an inline HTML element used to apply styles to specific parts of text without changing the structure or meaning, but this tag doesn't provide the same semantic value as other tags like paragraphs.

  • You should use aria-hidden="true" and focusable="false" in an SVG to improve accessibility. These attributes are essential to provide a better experience for users with disabilities, as they indicate that the SVG is decorative or non-interactive.

  I hope you find it useful! 😄

  Happy coding!

  11

  Marked as helpful
• 

  Renukta-21•130

  @Renukta-21

  Posted over 1 year ago

  Saw your code, and I think you could simplify much more your styles, also you can check the repeated styles that are in the same file, noticed you have tha same media-query distributed all along your file for different elements, I'm a beginner to btw, congrats!

  11

  Marked as helpful