Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Testimonial: css grid, BEM

P
yoyov51234 240

@yoyov51234

Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


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

I'm proud of using the BEM . I learned minmax for grid, but haven't got a closer look at the grid basics... will submit the solution again after get more concepts of the responsive grid

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

Learned to use BEM

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

Responsive page using grid.

Any suggestion is appreciated!

Community feedback

P

@CreativeLogic

Posted

Hi. Good Job. The typography needs a bit of work regarding line-height, font-sizes, and more to make them fit their card containers. The design calls for the card to be about the size of the content. For responsiveness, I tackled this challenge recently using grid-template-areas. This is simpler and more intuitive to use than having to deal with grid line numbers. In your current solution, if you open it up and slowly shrink your browser window, there is a point where the cards are being squished too much before switching to the next media query. The solution is to either adjust the media query so that it switches before it gets squished, or to set a minimum width on the cards in your template-columns using minmax appropriately. You can use your browser dev tools to get the sizes.

You can do something like

@ media (desktop) { 'c-1 c-1 c-2 c-3' 'c-3 c-5 c-5 c-3' }

@ media (tablets) {  Here, I did a two column solution) 'c-1 c-1  'c-2 c-4' 'c-3 c-3' 'c-5 c-5' }

And for mobile I did the typical one column solution: @ media (mobile) {

'c-1' 'c-2' 'c-3' 'c-4' 'c-5' }

You can see my solution on my page.

Marked as helpful

0

P
yoyov51234 240

@yoyov51234

Posted

@CreativeLogic

Thanks~ I'm currently training myself on the grid items, will definitely check on your solution and learn from it.

I have a question, so the design file only gives two types of designs, mobile and desktop, for the medium size, how do we know when to divide to a new media?

0
P

@CreativeLogic

Posted

@yoyov51234 In your chrome dev tools there is a device icon nex to the arrow where you can click to inspect elements called toggle device toolbar. When you open this, it opens a new window with device widths on top. You can click on tablet size and see how your solution looks. However, when getting sizes, we don't just want to use a typical fixed size. We let the content or our solution dictate when we want to switch to a different layout. You slowly move your browser window to the left or right and see how the content looks. When your layout begins to cause issues, the device toolbar tells you the width and you can use this value. For media queries, it's best to use EM units. You can check out youtube for video guides on how to use this handy feature.

0
P

@CreativeLogic

Posted

@yoyov51234

Also, as a developer it's crucial we pay close attention to details. The design has a purplish border around Daniel and Patrick's icons. Then the qoutation mark in Daniel's card is supposed to be more to the left.

0

@AkoToSiJeromeEh

Posted

Hey ! Great work out there i just notice you using padding-right , padding-left , padding-right and padding-bottom in .container , i suggest to use shorthand for declaring a padding like that ex padding : 4em(top) 1.5em(right) 4em(bottom) 1.5em(left) or in you case padding : 4em(top and bottom) 1.5em(top and right) , its much cleaner and less code for me . that's all happy coding !!

Marked as helpful

0

P
yoyov51234 240

@yoyov51234

Posted

@AkoToSiJeromeEh Thanks Jeje, good advice. I didn't do the shorthand because I was using sass to calculate the PX to rem/em using a function and I wasn't sure if the function works that way or not. Will give it a try and let you know later.

0

@AkoToSiJeromeEh

Posted

@yoyov51234 alright man , happy coding

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord