Responsive, active, position, border, etc

Community feedback

• 

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  Hi! Luka, I checked your work bro and I noticed some issues and they are as following:

  1. Your card box-shadow is too much just update it with

  box-shadow: 0 15px 10px 0 rgb(19 23 26);
  

  it will make your design look better.

  2. Wrap your body content with main tag, it will solve your Report issue.

  3. Use width with relatibe units like %age to make card responsive rather than using absolute units like px like use added width: 313px;, so after this add margin auto and it will make it little bit more better alignment.

  4. Solve above and your mobile responsiveness will improved too.

  I hope you understand.

  Best Wishes

  Marked as helpful

  1
• 

  gabriele2k9•80

  @Gabry091Posted almost 3 years ago

  hey! I really like your solution but i want to give you a suggestion: when you hover the image (the image with the eye) make the eye have 1.0 opacity and add a transition, good job btw

  Marked as helpful

  1

  ---

  

  Luka•320

  @lukakavtarraPosted almost 3 years ago

  @Gabry091 yea i change that :) thanks for ur suggestion

  0