Grid

Community feedback

• 

  P

  rmmcfarlin•190

  @rmmcfarlinPosted about 2 months ago

  Your solution looks good, very close to the original design. Padding / margins and proportion of the image to the text section could be improved to get it pixel-perfect, but the solution still looks good and is close enough. Could add some semantic HTML for accessibility. Overall though, looks great!

  1

  ---

  

  Abenezer Haile•210

  @proAbenezerPosted about 2 months ago

  @rmmcfarlin thank you I will try to perfect the site

  0
• 

  Jay Khatri•4,250

  @khatri2002Posted about 2 months ago

  Hi @proAbenezer!

  The developed solution looks great!

  I just have a minor suggestion that you may find worth implementing:

  The drawers image appears to be slightly stretched, as it is unable to maintain its original aspect ratio. To fix this, consider adding the Tailwind class object-cover (object-fit: cover). This ensures that the image takes up all the available space while maintaining its aspect ratio by cropping it equally from both the right and left sides.

  However, upon reviewing the design reference, it appears that the image is only being cropped from the right side, not the left. To align with the design reference, you can also add the object-left class (object-position: left), which will ensure that the image is aligned properly.

  The rest of the solution looks great! Keep up the excellent work! 🚀

  0

  ---

  

  Abenezer Haile•210

  @proAbenezerPosted about 2 months ago

  @khatri2002 Thanks a lot for your suggestion! I implemented object-cover along with object-left, and it worked perfectly. Now the image maintains its aspect ratio while aligning correctly with the design reference. I really appreciate your keen eye for detail and your helpful feedback!

  Thanks again, and looking forward to more of your insights!

  1