HTML and CSS with flexBox

Community feedback

• 

  KonradOlech•55

  @KonradOlechPosted almost 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

  1

  ---

  

  Studio Webpic•40

  @StudioWebpicPosted almost 4 years ago

  @KonradOlech Thx you so mutch for your feedback ! really interesting and I will apply this for my next projects :)

  0
• 

  Elise Rousseau•685

  @SarahHenriettePosted almost 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

  1

  ---

  

  Studio Webpic•40

  @StudioWebpicPosted almost 4 years ago

  Hi, Thank you very much ! Your message is very encouraging and motivating :)

  0