HTML, CSS, Grid template

Community feedback

• 

  Agata Liberska•3,995

  @AgataLiberskaPosted over 3 years ago

  Hi @Marufjan! Nice work on this challenge, it looks great!

  I noticed two small things: your media queries go up to 500px on mobile, then the next is from 511px up = which means that between 501px and 510px we get very very narrow cards - it's an easy fix of course :)

  Also, I would add some padding on left and right to your container class, so that the whole thing doesn't 'touch' the edges of the screen on smaller devices.

  I would also maybe add an alt text to the images, even if it's just the name of the person? But you may want to look into it, maybe leaving it blank is a better option.

  Hope this helps :)

  Marked as helpful

  0

  ---

  

  Mamatkulov Maruf•340

  @MarufjanPosted over 3 years ago

  @AgataLiberska Hi Agata, nice to hear from you :) Oh, yes media queries indeed has that problem lol. maybe, it was a typo or lost of concentration. And Padding thing is also true;) Next time I work on those things. Appreciate it !

  0