All comments

• 

  Studio Webpic• 40

  @StudioWebpic

  Submitted about 3 years ago

  Hi,

  Second challenge with HTML / CSS abd Flexbox. Not easy to place the circels !

  Thx to everybody who let a feedback :)

  

  HTML and CSS with flexBox

  2

  4

  0

  

  KonradOlech• 55

  @KonradOlech

  Posted about 3 years ago

  Hi, my advice is deleting max-width attribute from .container, because it is messing with styling in wider screens (4k displays are more common every year) and it is unecessary.

  I would move "circles background" from .container and merge it with body bg. In hypothetical situation when you decide to move/delete/change profile card component your solution will change bg of website.

  Third advice - to fix this blue line between img and border (on person photo) add "background: #fff" attribute on img. It is caused by bad anti-aliasing on svg render so this way you cover it.

  Anyway - good job, very similar to my solution :D

  1

  This comment has received 1 replyShow