Pure HTML and CSS. It is a "Real junior work" but now incl. CSS GRID

Community feedback

• 

  Fidel Lim•2,775

  @fidellimPosted about 3 years ago

  Hi @ezequielmagi,

  Great job finishing the project! Looks great for both desktop and mobile views. Just a suggestion, if you're interested at copying the exact color of the color overlay of image, you can do this:

  .img-container {
      background-image: url(../img/image-header-mobile.jpg);
      background-color: var(--Soft-violet);
      background-blend-mode: multiply;
  }
  

  background-blend-mode: multiple darkens the color overlay of the image. You can remove the div for the color as well.

  I hope it helps :)

  Marked as helpful

  1

  ---

  

  ezequielmagi•60

  @ezequielmagiPosted about 3 years ago

  @fidellim GENIUS!!!! i did not about blend-mode!! i will improve it! Thanks!

  0

  

  Fidel Lim•2,775

  @fidellimPosted about 3 years ago

  @ezequielmagi That alright! Learning never stops :) Good luck!

  Marked as helpful

  1
• 

  KingOfTheFlies•110

  @ezraakankwasaPosted about 3 years ago

  You did such a great job with your layout. I'd advise that you keep all your CSS in one file and use media queries to target the different screen sizes.

  Marked as helpful

  1

  ---

  

  ezequielmagi•60

  @ezequielmagiPosted about 3 years ago

  @ezraakankwasa Thank you very much! That's what I was originally doing. Then watching some courses and youtube videos they recommend, for accessibility and loading speed, to separate it. I will follow your advice for the next challenge!

  0

  

  KingOfTheFlies•110

  @ezraakankwasaPosted about 3 years ago

  @ezequielmagi glad to help

  Marked as helpful

  0