All comments

• 

  raz hershko• 100

  @razh23398

  Submitted almost 2 years ago

  I had many difficulties with the placement of the cubes. In addition, it was very difficult for me to make the hover for the main image. Until now I can't figure out how to make the icon of the eye white.

  

  I had difficulties with the hover main img color

  1

  30

  

  xmat• 160

  @mati0165

  Posted almost 2 years ago

  Hi, i will tell you how to change the icon of eye to white. U have it white, but u change the opacity of every element in your div, so the opacity is also on the eye. For fix the problem with eye not being white there is an easy solution

  • For first u have to your background be transparent, not div as u had in your code, for that u will have to add 50% as ur 4th value to hsl:

  .overlay {
  background-color: hsl(178, 100%, 50%, 50%);
  } 
  

  • then u change ur opacity at hover to 1

  .container:hover .overlay {
    opacity: 1;
  }
  

  I think that solved your problem.

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Yathish• 50

  @yathishg

  Submitted almost 2 years ago

  Hello everyone, Accomplished second challenge. pls let me know if any mistake in the code and any suggestions. Thank you

  

  3-column-preview-card

  1

  30

  

  xmat• 160

  @mati0165

  Posted almost 2 years ago

  Hi, nice that you have done that challenge, but i have some suggestions for you:

  • The best way to create a page is using divs not table like you do in that challenge. Also for that i would prefer using grid.
  • The page is not centered, but the best way to do that is when u use flexbox and add in css code min-height: 100vh for your body.

  Marked as helpful

  0

  This comment has received 3 repliesShow