Profile card component using css flebox for the layout of the website

Community feedback

• 

  Ctrl+FJ•810

  @FlorianJourdePosted about 3 years ago

  Ok, I'll just steal your background-position properties ! It's so good that you don't even need media queries !

  Marked as helpful

  0

  ---

  

  Martha•510

  @Mtc-21Posted about 3 years ago

  @FlorianJourde hi, thanks a lot, there is media query for min-width:1440 which is for desktop, I suggest you the following link for background-position, it helped me a lot -> https://developer.mozilla.org/en-US/docs/Web/CSS/background-position developer.mozilla is great for anything.

  0
• 

  Martin de Boer•50

  @martindb1988Posted about 3 years ago

  Good job on this one, if i can make a recommendation look into semantic html. Makes the html better readable.

  Marked as helpful

  0
• 

  Fidel Lim•2,775

  @fidellimPosted about 3 years ago

  Hi @Mtc-21,

  Great job finishing the project! It looks great on desktop and mobile devices. There isn't much to improve I would say. It would be great to show your code in your Github repository. So others can have a look at it. We could probably suggest some points regarding your code. :)

  Marked as helpful

  0
• 

  CyrusKabir•1,885

  @CyrusKabirPosted about 3 years ago

  hello my dear friend ♥ it's good and clean but just one thing :

  • increase font-size just a little 1 rem for heading it's good and 0.8 rem for category text (number below)

  Marked as helpful

  0