Profile card

Community feedback

• 

  P

  visualdennis•8,375

  @visualdennissPosted over 1 year ago

  Hello Tinker,

  nice job with the challenge and also with the background images! Few suggestions for further improvement:

  • background-color does not seem to match the design, u can use this: background-color: #19a2ae;

  • you dont need height: auto; for the .card-container, as height property has default value of auto already. Similarly you don't need width: 100vw for the main. Also it is better to replace height: 100vh with min-height: 100vh to not to set any constraints.

  • the divider line seems to be too thick, you can simply remove the border on it.

  • The about text content (likes, followers) don't seem to be aligned in the center due to uneven paddings.

  Hope you find this feedback helpful!

  Marked as helpful

  1

  ---

  

  Taruna•305

  @Tiyana19Posted over 1 year ago

  @visualdenniss Thank you so much about the divider just want to ask should remove only border-top or whole border and then give it a height ?

  1

  

  P

  visualdennis•8,375

  @visualdennissPosted over 1 year ago

  @Tiyana19 It already looks much better, so no need to do anything else in my opinion, what you did is good enough.

  Marked as helpful

  1