@Drrecommended
@LShiznit
Submitted
Difficult to rely only by responsive means. Using grid gets complicated when you revisit it. Fixing images with absolute was also tedious, as well as the typography.
I feel the code could have been simplified if I had strategized for responsiveness. Padding was a little inconsistent.
Couldn't get that first curve to stay fixed relative to the phrase. I'm not sure how to do that.
@Drrecommended
Posted
Hi your solution looks great! I did notice that your <p> spills over the image. I would suggest setting a max width on this. I would also suggest fixing your typography. You can also use opacity to slightly lighten up the <p>. If you look at the bottom of your page it looks like there is a margin that you don't want. Try setting the margin bottom to 0 on that element. Also look more into margins and paddings. You can also use grid to space things out in your header as needed. Cheer!
@hozayves
Submitted
Hello friends I'm beginner in CSS, I just need your feedback in my code firstly, check BEM standard, HTML structure, CSS. Thanks for your good response
@Drrecommended
Posted
Looks great! You're doing very well. Your site needs to be responsive though and the shapes in the background are becoming squished. I recently took this 20-day challenge course by Kevin Powell and I think it would really help you! I learned so much from this and it really leveled up my skills. It's also FREE! Check it out!
https://courses.kevinpowell.co/courses/conquering-responsive-layouts
@surphury
Submitted
Feel free to correct me, ask me... Due to that I can improve my web development skills... Thanks beforehand
@Drrecommended
Posted
Looks great. I do see however some spacing issues with the padding on the bottom of "we only provide great products" and the bottom of the picture in the user review boxes
Marked as helpful
