Samuel Fuchs• 60
@samuelfuchs
Submitted
This is my second challenge. Thank you for all of this. Cheers from Brazil
Cedrick Garcia
@cedgarcia
All comments
- Cedrick Garcia• 905
@cedgarcia
Posted
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)
2 - Aldair Avalos• 50
@AMLO-Bot
Submitted
I would love having feedback about the implementation of BEM arquitecture for this project. I reaaly think there is a lot of space for improvement and Im just a newbie so I do not know really what Im doing. Appreciate in advance your comments
Cedrick Garcia• 905@cedgarcia
Posted
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
2 - Massimiliano Cornale• 350
@mcornale
Submitted
Hi everybody!
This is my first challenge.
Any advice is welcome. 🐙
Cedrick Garcia• 905@cedgarcia
Posted
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
0 - oodana• 70
@oliferosadana
Submitted
finally i completed all newbe challenge in html & css
Cedrick Garcia• 905@cedgarcia
Posted
Good job. Your code looks clean. If you want to learn more, kindly follow my github account You can see my solutions in there
0 - Wiktor• 20
@wiktorpagacz
Submitted
This is my first challenge on this page and my first responsive design. I'd love to get some feedback and advice
Cedrick Garcia• 905@cedgarcia
Posted
Good Job on your first challenge. Try to use Relative Units e.g. (vh,vw, rem ,em) for a better responsive design
1 - Tae Ishida• 35
@taeishida
Submitted
Here goes my first challenge ever. Any comments are welcomed and appreciated.
Cedrick Garcia• 905@cedgarcia
Posted
Good job. Your code is clean. Keep it up
0 - Robin• 30
@Robinkariuki
Submitted
Hi everyone,
First challenge attempted, i'm open to any feedback on how to improve my approach with this current challenge.
cheers!
Cedrick Garcia• 905@cedgarcia
Posted
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); }
5 - Jamie Wales• 90
@Jamie-Wales
Submitted
Only just started learning front-end development after a few years of being in a client services team, looking to jump over so feed appreciated :D
Cedrick Garcia• 905@cedgarcia
Posted
Good Job, Positioning of background is always hard. Your code is clean Just deep dive on media queries to make it responsive
4 - Guilherme Silva• 140
@silva-guilherme
Submitted
Any improvement sugestions will be very appreciate! thx
Cedrick Garcia• 905@cedgarcia
Posted
Good job. Your code is clean. Just remove the "scrset" (<img src=
images/icon-supervisor.svgalt=Calculator Iconsrcset=``>)4 - Eskender• 65
@egxperience
Submitted
All kind of feedback highly appreciated as initially didn't get idea how to start the project. All job done mixing vanilla CSS and Bootstap codes and would like to get feedback on "photo" and "stats" code (it was not easy to make it working at the beginning)
Cedrick Garcia• 905@cedgarcia
Posted
Good work! You just need to adjust the bottom-background to right.
4 - Yagiz• 140
@tosbaa
Submitted
Hello, this is my first example that i solved for this website. I'm waiting for the feedbacks. Thank you !!
Cedrick Garcia• 905@cedgarcia
Posted
Try to resize your Card, and follow the colors given by the guide. Your code looks clean. Specially the profile-stats
4 - Raihan-21• 210
@Raihan-21
Submitted
i appreciate if there is any feedback to make my code more effective thanks
Cedrick Garcia• 905@cedgarcia
Posted
Try to separate your css code from your html. Other than that, your code looks clean
5 - Harielson Mendes• 45
@hbmendes
Submitted
Challenge using HTML, CSS, my knowledge in Front-end beginner.
Cedrick Garcia• 905@cedgarcia
Posted
This comment was deleted
3 - Anthony• 10
@akoshmanov
Submitted
Hello, everyone! I'm a rookie in that whole sphere, so do not wait a lot. I didn't do the mobile version because I have no idea how even. I found several colors, but couldn't get why there were three of them so I used only one, which was not grey ( another two ones were). I am not sure that my code is right, but I would appreciate your helping and notice! Thank you. P.S. I don't know why, but if you check my code you'll see, that I changed display properties for 2 div tags (sphere_1, sphere_2) to inline-block, but it hadn't been applied. Why?
Cedrick Garcia• 905@cedgarcia
Posted
Add an overflow-x:hidden in your body in css;. To make it responsive(mobile) add media queries.Your code looks good
3
