HTML and CSS with flexBox

Community feedback

• 

  KonradOlech•55

  @KonradOlech

  Posted about 4 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

  11
• 

  Elise Rousseau•685

  @SarahHenriette

  Posted over 4 years ago

  Hi Studio Webpic , 🙂

  Well done because it's really well done and the visual is almost identical 👍

  yes I agree enough to place the circles it is not obvious, I had also had a little trouble especially on the mobile

  Good continuation

  11