I used flex for this project and i am not happy with this project. :(

Community feedback

• 

  Adriano•38,410

  @AdrianoEscarabotePosted over 2 years ago

  Hello everything is fine? I liked your project, you did a good job.

  Don't be discouraged, we all have difficulties and I know that if you put a little more effort into improving what's making you upset, you'll feel better.

  I'll give you some tips:

  1- This is a project that would be better if you had used grid, I saw that you used flex and it wasn't bad.

  Some changes I made to the code:

  .section-1{ width: 500px; } main { width: 100%; padding: 0 20px; }

  And I took the media query from 940px.

  Made the layout a little more similar to the example.

  The rest was very good. I hope it helps!.. 👍

  Marked as helpful

  0
• 

  Enmanuel Otero Montano•2,155

  @Enmanuel-Otero-MontanoPosted over 2 years ago

  Hello ayush!

  You must say what your doubt is or what you need help with in order to give you the best possible help.

  Marked as helpful

  0
• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Ayush, congratulations for your new solution!

  I saw your solution and I think you've done a great work here, you can improve it even more,

  For example, you can improve the overlay color and the image responsivity with two easy steps:

  Make the image have the same purple overlay effect, matching the design files you can use mix-blend-mode. See the code below

  img {
  mix-blend-mode: multiply;
  opacity: 75%;}
  

  Improve the image responsiveness, adding display: blockand max-width: 100% and object-fit: cover to make the image auto-crop when resizing inside the column:

  img {
      display: block;
      object-fit: cover;
      max-width: 100%;
  }
  

  👋 I hope this helps you and happy coding!

  0