All comments

• 

  Martha•510

  @Mtc-21Submitted about 3 years ago

  What could be the possible improvements?

  

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

  4

  20
  

  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
• 

  Sofía Pérez Mantilla•80

  @sofiaperezmSubmitted about 3 years ago

  I would like to get advice on how to avoid repetitive code in CSS. (I am aware of the BEM theory, but I struggle in the practice). Thanks in advance :)

  

  Mobile-first job listings with filters using Vanilla Javascript

  3

  10
  

  Martin de Boer•50

  @martindb1988Posted about 3 years ago

  CSS works from top to bottom so once a style is defined the DRY method comes in place. Good use of CSS variables but i would look into your class and id naming with __ and — i found that very confusing.

  Marked as helpful

  0

  This comment has received 4 repliesShow
• 

  Joe sherbini•20

  @yousefshrpinySubmitted about 3 years ago

  feel free to add any edition or any recommendation for a cleaner code

  

  Frontend-Mentor---Profile-card-component with flexbox

  3

  10
  

  Martin de Boer•50

  @martindb1988Posted about 3 years ago

  Great job, but a few suggestions. Naming your css file css.css is rather weird also in terms of clean code your css has tabs and no tabs.

  Try to be more consistent with that, clean code makes the code better readable for other devs.

  2

  This comment has received 2 repliesShow