Vanilla JS and SASS

Community feedback

• Account deleted

  good effort i have some suggestions for you,

  the heading part is to big maybe something like from 1.3 to 2 em is good, and the header section try to give it some padding maybe something like 100px it will be better and the card font size i think it needs to be more bigger and the body background color i think it sholud be more darker like this: #1e1f29

  and finaly thanks for read my comment and iam if my english is bad

  0

  ---

  

  dportillo23•260

  @dportillo23Posted over 3 years ago

  @Abdulrahman-M-Darwish Thank you very much for your feedback and suggestions, I will take them into account.

  Thank you for taking the time to review my solution and give me some feedback.

  0
• 

  ibrahim•450

  @bagggaryPosted over 3 years ago

  REALLY COOL ! JUST TRY TO CHANGE THE FONT SIZE MAKE IT LITTLE BIGGER , AND ALSO SOCIAL ICONS LITTLE UP. KEEP THE GOOD WORK 😃

  0

  ---

  

  dportillo23•260

  @dportillo23Posted over 3 years ago

  @ENG-SKYLER thank you very much, I appreciate your feedback. I will make those modifications you mentioned. Thank you for your words 😊

  0