All comments

• 

  Yusaf Saddiq• 20

  @YusafS94

  Submitted over 1 year ago

  This is the first F.E.M project i have completed. Eager to hear feedback on how I can improve, where i went wrong and how i could do better. I tracked how long it took me to complete this challenge using a simple time sheet on Excel.

  Feel free to be honest with your criticisms 🙂

  Tools used:

  • HTML
  • Vanilla CSS
  • VSCode editor

  Time taken to complete: 4 hours 19 minutes

  

  Four Card Feature Section

  1

  2

  0

  

  Juan Carlos• 255

  @JCruz-Dev

  Posted over 1 year ago

  Nice Job! Just take care of the warnings about the accessibility. In this case, seems it is about a heading you put over there. Check that you are using the correct heading levels, like including and H2 tag when its is needed and H3 under and H2 tag is needed and so on.

  Keep it going!

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Waleed OMEGA• 70

  @WaleedOMEGA

  Submitted over 3 years ago

  

  Article Preview Component

  1

  1

  0

  

  Juan Carlos• 255

  @JCruz-Dev

  Posted over 3 years ago

  You got it to some point. That is good!

  Things that you have to improve are:

  • Watch for the layout on mobile
  • Look into more the style guide to get the close possible to the fonts, spacing and more.

  0
• 

  Bogdan Shchavinskyi• 50

  @master-bogdan

  Submitted over 3 years ago

  

  Article preview component with JS

  1

  2

  0

  

  Juan Carlos• 255

  @JCruz-Dev

  Posted over 3 years ago

  Good work. You have to improve the following:

  • Watch the responsive on mobile
  • Tooltip is appearing on Mobile, should appear only in desktop.
  • Add an animation for the tooltip to get a better experience because is coming to hard

  After those points. Nice Work!

  0

  This comment has received 1 replyShow
• 

  Maria• 60

  @Maria131621

  Submitted over 3 years ago

  

  Fylo data storage using flexbox & Sass

  2

  5

  0

  

  Juan Carlos• 255

  @JCruz-Dev

  Posted over 3 years ago

  Nice work and smooth animation. You should fix the HTML issue and that would be fine :)

  In your github repo, try to deploy the site with netlify or vercel and put the link in the README.md so anyone who only visit yout github profile can watch your work too.

  0

  This comment has received 1 replyShow
• 

  Juliane Gaudencio• 215

  @julianegaudencio

  Submitted over 3 years ago

  I still have difficulty in the responsive part so it only applies the difference less than 890px, I'm starting the challenges now so some things I needed to research because it went blank, please leave your opinions.

  

  responsive website with pure css

  2

  5

  1

  

  Juan Carlos• 255

  @JCruz-Dev

  Posted over 3 years ago

  Well done!

  0

  This comment has received 1 replyShow
• 

  Juan Carlos• 255

  @JCruz-Dev

  Submitted over 3 years ago

  

  Fylo data storage component

  2

  3

  0

  

  Juan Carlos• 255

  @JCruz-Dev

  Posted over 3 years ago

  I added the animation on the bar and Gigabyte left title. Seems that frontend mentor takes the capture of the first rendered elements. You can see the fullsite view on the preview site button I completed it in its entirely.

  0

  This comment has received 2 repliesShow
• 

  vtejaeta• 70

  @vtejaeta

  Submitted almost 4 years ago

  Any feedback would be appreciated!!

  

  Single price grid component

  1

  2

  0

  

  Juan Carlos• 255

  @JCruz-Dev

  Posted almost 4 years ago

  Nice work. Some things that you should improve are:

  A) The box-shadow you have in your solution should be smoother. The one you have in the card should be fixed to be closer to the design and the one in the button too.

  B) The font sizes try to get it closer to the design itself.

  C) Review the Report, seems you have duplicate ID in your HTML

  2

  This comment has received 2 repliesShow