Frontend Mentor | jkaps9's profile

bobyanov•100

@bobyanov

Submitted 9 months ago

testimonial grid section

1

jkaps9•140

@jkaps9

Posted 9 months ago

This looks decent, but you have some work to do to match the design.

It is snapping to the mobile layout at too wide of a screen size - you should set max-width closer to 768px instead of the 1500px you have currently.
The quote text on the white background is so low that it is hard to read, you should up the opacity to 70%. Check the style guide for recommended opacity.
In your CSS you should consider using custom properties (variables) for the colors which are used throughout the project.

WaelBenseghir•160

@WaelBenseghir

Submitted 9 months ago

four-cards-grid-design

1

jkaps9•140

@jkaps9

Posted 9 months ago

This looks really good!

I like your use of variables in the CSS for the font size, family, and weights. I'm not very familiar with CSS Grid, but seeing your code showing the placement helped me learned something new!

Ayomide philip•180

@Ayomide-Philip

Submitted 10 months ago

What are you most proud of, and what would you do differently next time?

Using Grid and Flex together

What challenges did you encounter, and how did you overcome them?

I encounter a challenge when using grid

What specific areas of your project would you like help with?

@media query

Product Preview Card Componet Main

1

jkaps9•140

@jkaps9

Posted 10 months ago

Looks great!

DolbyTheSheep•120

@DolbyTheSheep

Submitted 10 months ago

What challenges did you encounter, and how did you overcome them?

The difficult part was drawing the lines in the table. I also got stuck a bit in the mobile layout, thinking how to resize the image over the padding of the div. Finally I added a new div and the problem was resolved :)