Cedrick Garcia
@cedgarciaAll comments
- @samuelfuchs@cedgarcia
Hi, Samuel Fuch!
Well done on this challenge! Your code is clean. I suggest you to add a max-width property in your main to make it look better(it stretches too much on a bigger screen ). Overall Goodjob. (add max-width:1400px; in your main)
- @AMLO-Bot@cedgarcia
As a newbie, your code looks great. Try to minimize your usage of a double dash(--) in naming classes because it might make someone confuse, and your CSS code will look messier(especially if you will use variables in CSS). You can check my solution here: (https://cedgarcia-front-end-mentor-solutions.vercel.app/1_profile-card-component-main/index.html), GitHub repo(https://github.com/CedGarcia/FrontEndMentorSolutions/tree/master/1_profile-card-component-main). Overall Good Work
- @mcornale@cedgarcia
Good Job. Your design looks perfect and your code is clean. Just try to separate your css code next time. Kindly follow me on github. Thank you
- @oliferosadana@cedgarcia
Good job. Your code looks clean. If you want to learn more, kindly follow my github account You can see my solutions in there
- @wiktorpagacz@cedgarcia
Good Job on your first challenge. Try to use Relative Units e.g. (vh,vw, rem ,em) for a better responsive design
- @taeishida@cedgarcia
Good job. Your code is clean. Keep it up
- @Robinkariuki@cedgarcia
Good job on your first attempt. Just add a "background-repeat:no -repeat;" on your styling. It seems that you did'nt use the "bg-pattern-bottom.svg". Try to add it by doing this:
body { background-image: url(images/bg-pattern-top.svg), url(images/bg-pattern-bottom.svg); background-repeat: no-repeat; background-size: 800px; background-color: rgb(0, 182, 199); }
- @Jamie-Wales@cedgarcia
Good Job, Positioning of background is always hard. Your code is clean Just deep dive on media queries to make it responsive
- @glmsilva@cedgarcia
Good job. Your code is clean. Just remove the "scrset" (<img src=
images/icon-supervisor.svg
alt=Calculator Icon
srcset=``>) - @egxperience@cedgarcia
Good work! You just need to adjust the bottom-background to right.
- @tosbaa@cedgarcia
Try to resize your Card, and follow the colors given by the guide. Your code looks clean. Specially the profile-stats
- @Raihan-21@cedgarcia
Try to separate your css code from your html. Other than that, your code looks clean
- @akoshmanov@cedgarcia
Add an overflow-x:hidden in your body in css;. To make it responsive(mobile) add media queries.Your code looks good