3-Preview-Card-Component using HTML and CSS

Community feedback

• 

  Jalga•180

  @coder-96Posted 12 months ago

  Merhaba, @aybegu! Mobil çözünürlükte sütun olarak görünmesini deneyebilirsiniz.

  The rest looks fine!

  1

  ---

  

  Ayça Gürler•180

  @aybeguPosted 12 months ago

  @coder-96 Thank you! Will do that!

  0
• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 12 months ago

  Hello again @aybegu! 😊

  Your project looks very good!

  I have a few suggestions for you to improve it:

  • Try using semantic HTML a bit more. For example, I noticed that you used 2 div tags for the main container. You could've used only one, and since it's the main content, it could be the main tag. Using divs is not wrong, but since they don't have semantic value, they are recommended for smaller elements.

  • I also noticed that you started the project by it's desktop version and you don't have a mobile version yet. Try the mobile first approach. It makes a lot easier to create the desktop version once the mobile one is finished. The other way around is frequently more difficult.

  Other than those little things, you did a great job!

  I hope it helps!

  0

  ---

  

  Ayça Gürler•180

  @aybeguPosted 12 months ago

  @danielmrz-dev Thank you Daniel! Yes I always forget to do the mobile version :D And I will pay attention to the HTML more next time!

  1